Bug #35770 for RT-Client-REST: t/80-timeout hangs on Windows

Sun May 11 09:34:27 2008 dagolden [...] cpan.org - Ticket created

Subject:	t/80-timeout hangs on Windows
Date:	Sun, 11 May 2008 09:34:10 -0400
To:	bug-RT-Client-REST [...] rt.cpan.org
From:	"David Golden" <dagolden [...] cpan.org>

On RT-Client-Rest-0.36, the t/80-timout.t file hangs. (Are you using alarm? This isn't reliable on Windows.) David

Mon May 12 05:09:27 2008 DAMS [...] cpan.org - Correspondence added

We are using LWP::UserAgent to do the HTTP connection. The timeout option is handled by this module directly. It turns out that the simple timeout option is not properly supported under windows, as LWP::UA seems to be using alarms to do it. I'll try to test the workaround proposed in the lwp FAQ, as soon as I can get a hand on a windows perl. Thanks for the report

Mon May 12 05:09:35 2008 The RT System itself - Status changed from 'new' to 'open'

Mon May 12 09:22:03 2008 dagolden [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #35770] t/80-timeout hangs on Windows
Date:	Mon, 12 May 2008 09:21:52 -0400
To:	bug-RT-Client-REST [...] rt.cpan.org
From:	"David Golden" <dagolden [...] cpan.org>

On Mon, May 12, 2008 at 5:09 AM, Damien Krotkine via RT > We are using LWP::UserAgent to do the HTTP connection. The timeout Show quoted text

> option is handled by this module directly. It turns out that the simple > timeout option is not properly supported under windows, as LWP::UA seems > to be using alarms to do it. > > I'll try to test the workaround proposed in the lwp FAQ, as soon as I > can get a hand on a windows perl.

It's LWP::UA? That's annoying. It should be more portable. That's probably the cause of any number of Windows test failures. I'll go harass them next. Thanks! David

Tue Jan 27 23:25:21 2009 jpierce [...] cpan.org - Status changed from 'open' to 'stalled'

Tue Jun 15 22:06:44 2010 bobtfish [...] bobtfish.net - Correspondence added

Did this get anywhere? Was the issue in LWP? Can we close this? Thanks in advance. t0m

Tue Jun 15 22:06:46 2010 The RT System itself - Status changed from 'stalled' to 'open'

Tue Jun 15 22:34:19 2010 dagolden [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #35770] t/80-timeout hangs on Windows
Date:	Tue, 15 Jun 2010 22:33:49 -0400
To:	bug-RT-Client-REST [...] rt.cpan.org
From:	David Golden <dagolden [...] cpan.org>

On Tue, Jun 15, 2010 at 10:06 PM, Tomas Doran via RT <bug-RT-Client-REST@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=35770 > > > Did this get anywhere? Was the issue in LWP? Can we close this?

I don't know that it was ever fixed, but haven't looked at the issue in ages. Feel free to close.

Wed Jun 16 05:16:34 2010 bobtfish [...] bobtfish.net - Status changed from 'open' to 'resolved'

Thu Aug 11 08:32:58 2011 CHORNY [...] cpan.org - Correspondence added

Still hangs in version 0.41. I tried running server and client from this test manually, as separate processes. I see 3 errors like this: not ok 3 - The object isa RT::Client::REST::RequestTimedOutException # Failed test 'The object isa RT::Client::REST::RequestTimedOutException' # at 80-timeout.t line 31. # The object isn't a 'RT::Client::REST::RequestTimedOutException' it's a 'RT::Client::REST::HTTPException' 1-st test passes, but server exits after it. -- Alexandr Ciornii, http://chorny.net

Thu Aug 11 08:32:59 2011 The RT System itself - Status changed from 'resolved' to 'open'

Thu Aug 11 08:34:57 2011 CHORNY [...] cpan.org - Correspondence added

IMHO, this test should be skipped on Windows, obviously fork and alarm does not work together on Windows. -- Alexandr Ciornii, http://chorny.net

Thu Aug 11 08:35:33 2011 CHORNY [...] cpan.org - Cc CHORNY added

Mon Jan 09 13:21:18 2012 jlmartinez [...] capside.com - Taken

Mon Jan 09 13:21:49 2012 jlmartinez [...] capside.com - Correspondence added

Hi, This behaviour has been correted in RT::Client::REST 0.43. Thanks for contributing. JLMARTIN

Mon Jan 09 13:21:50 2012 jlmartinez [...] capside.com - Status changed from 'open' to 'resolved'