Bug #37151 for SOAP-Lite: Strawberry perl 5.10 - Tests fail and then later hang

Thu Jun 26 20:14:47 2008 toddr [...] cpanel.net - Ticket created

Subject:

Strawberry perl 5.10 - Tests fail and then later hang

Message body is not shown because it is too large.

Thu Jun 26 20:21:11 2008 toddr [...] cpanel.net - Correspondence added

From:

todd.e.rinaldo [...] jpmorgan.com

Hi, I've added some tests to CGI.t to see where it's hanging. The actual call for no warnings seems to be going wacky? Show quoted text

--- OUTPUT --- C:\strawberry\cpan\build\SOAP-Lite-0.710.07-hBBQQX>prove -v -b t\SOAP\Transport\HTTP\CGI.t t\SOAP\Transport\HTTP\CGI.... ok 1 - use SOAP::Lite; # send_recieve called ok 2 - Object defined

---- HERE is the code the output is coming from --- use Test::More qw(no_plan); use_ok('SOAP::Lite'); # +trace; my $soap = SOAP::Lite->new()->proxy('http://'); ok($soap, "Object defined"); no warnings qw(redefine once); diag("HERE"); *SOAP::Transport::HTTP::Client::send_receive = \&SOAP::Transport::CGI_TEST::Client::send_receive; diag "send_recieve called";

Thu Jun 26 20:22:51 2008 toddr [...] cpanel.net - Correspondence added

From:

todd.e.rinaldo [...] jpmorgan.com

Hi, I've added some tests to CGI.t to see where it's hanging. The actual call for "my $som = $soap->call('test');" seems to be failing? Show quoted text

--- OUTPUT --- C:\strawberry\cpan\build\SOAP-Lite-0.710.07-hBBQQX>prove -v -b t\SOAP\Transport\HTTP\CGI.t t\SOAP\Transport\HTTP\CGI.... ok 1 - use SOAP::Lite; # send_recieve called ok 2 - Object defined

---- HERE is the code the output is coming from --- use Test::More qw(no_plan); use_ok('SOAP::Lite'); # +trace; my $soap = SOAP::Lite->new()->proxy('http://'); ok($soap, "Object defined"); no warnings qw(redefine once); diag("HERE"); *SOAP::Transport::HTTP::Client::send_receive = \&SOAP::Transport::CGI_TEST::Client::send_receive; diag "send_recieve called";

Thu Jun 26 20:25:02 2008 toddr [...] cpanel.net - Correspondence added

From:

todd.e.rinaldo [...] jpmorgan.com

my bad, to be clear, the source of the hang appears to be the below code... my $som = $soap->call('test');

Sat Jun 28 12:45:45 2008 martin.kutter [...] fen-net.de - Correspondence added

Subject:	Re: [rt.cpan.org #37151] Strawberry perl 5.10 - Tests fail and then later hang
Date:	Sat, 28 Jun 2008 18:45:16 +0200
To:	bug-SOAP-Lite [...] rt.cpan.org
From:	Martin Kutter <martin.kutter [...] fen-net.de>

CGI.t seems to hang due to encoding issues - the test server (which emulates CGI by using open2) returns a string containing "Überall" - looks like it sends the content-length correctly (8 bytes), but sends the data in cp1252 (7 bytes). The failing test is due to SOAP::Transport::IO requiring the POSIX macro EWOULDBLOCK, which apparently is not available on Windows. I'd suggest the following resolution: a) fix the transport or CGI test encoding issues b) skip the SOAP::Transport::IO test if EWOULDBLOCK is not available, and open a bug report for this behaviour (might take some time, and I think SOAP::Transport::IO is rarely used). I'd appreciate any help, as I only have limited access to a windows box for testing. Thanks for reporting, Martin Am Donnerstag, den 26.06.2008, 20:21 -0400 schrieb Todd E. Rinaldo via RT: Show quoted text

> Queue: SOAP-Lite > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=37151 > > > Hi, I've added some tests to CGI.t to see where it's hanging. The > actual call for no warnings seems to be going wacky? > > --- OUTPUT --- > > C:\strawberry\cpan\build\SOAP-Lite-0.710.07-hBBQQX>prove -v -b > t\SOAP\Transport\HTTP\CGI.t > t\SOAP\Transport\HTTP\CGI.... > ok 1 - use SOAP::Lite; > # send_recieve called > ok 2 - Object defined > > > ---- HERE is the code the output is coming from --- > use Test::More qw(no_plan); > use_ok('SOAP::Lite'); # +trace; > my $soap = SOAP::Lite->new()->proxy('http://'); > ok($soap, "Object defined"); > no warnings qw(redefine once); > diag("HERE"); > *SOAP::Transport::HTTP::Client::send_receive = > \&SOAP::Transport::CGI_TEST::Client::send_receive; > diag "send_recieve called"; >

Sat Jun 28 12:45:46 2008 The RT System itself - Status changed from 'new' to 'open'

Mon Jun 30 10:20:44 2008 toddr [...] cpanel.net - Correspondence added

From:

todd.e.rinaldo [...] jpmorgan.com

Martin, Thanks for checking on this. I guess I feel a little grey on how to respond. 1. It makes no sense that the POSIX macro would be unavailable if it's a part of perl. If it's a part of the OS, then can we leverage the POSIX module? 2. As for testing for then skipping these tests, how does this break functionality that this doesn't work? Would it hang in real life? 3. I'm happy to help debug windows. Please let me know how I can help. If this needs to go offline, feel free to email me directly. Todd On Sat Jun 28 12:45:45 2008, martin.kutter@fen-net.de wrote: Show quoted text

> CGI.t seems to hang due to encoding issues - the test server (which > emulates CGI by using open2) returns a string containing "Überall" - > looks like it sends the content-length correctly (8 bytes), but sends > the data in cp1252 (7 bytes). > > The failing test is due to SOAP::Transport::IO requiring the POSIX

macro Show quoted text

> EWOULDBLOCK, which apparently is not available on Windows. > > I'd suggest the following resolution: > > a) fix the transport or CGI test encoding issues > b) skip the SOAP::Transport::IO test if EWOULDBLOCK is not available, > and open a bug report for this behaviour (might take some time, and I > think SOAP::Transport::IO is rarely used). > > I'd appreciate any help, as I only have limited access to a windows

box Show quoted text

> for testing. > > Thanks for reporting, > > Martin

Mon Jun 30 15:10:59 2008 martin.kutter [...] fen-net.de - Correspondence added

Subject:	Re: [rt.cpan.org #37151] Strawberry perl 5.10 - Tests fail and then later hang
Date:	Mon, 30 Jun 2008 21:08:35 +0200
To:	bug-SOAP-Lite [...] rt.cpan.org
From:	Martin Kutter <martin.kutter [...] fen-net.de>

Hi Todd, 1. I think EWOULDBLOCK is not available on windows (but I'm not sure). There are workarounds for doing non-blocking I/O on windows, but I haven't investigated... 2. Skipping the tests would of course leave SOAP::Transport::IO unusable on platforms without EWOULDBLOCK. However, direct IO transport is scarcely used. 3. ... any help appreciated :-) I think the hanging CGI.t could be a encoding issue, where the test script expects to read more bytes from the test server via STDIN than this sends. I've added a alarm() call to the test, so at least it should no longer hang (at https://soaplite.svn.sourceforge.net/svnroot/soaplite/branches/0.71/t/SOAP/Transport/HTTP/CGI.t ) I'd be very happy if you could confirm this guess... Martin Am Montag, den 30.06.2008, 10:21 -0400 schrieb Todd E. Rinaldo via RT: Show quoted text

> Queue: SOAP-Lite > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=37151 > > > Martin, > > Thanks for checking on this. I guess I feel a little grey on how to > respond. > > 1. It makes no sense that the POSIX macro would be unavailable if it's > a part of perl. If it's a part of the OS, then can we leverage the > POSIX module? > > 2. As for testing for then skipping these tests, how does this break > functionality that this doesn't work? Would it hang in real life? > > 3. I'm happy to help debug windows. Please let me know how I can help. > If this needs to go offline, feel free to email me directly. > > Todd > > On Sat Jun 28 12:45:45 2008, martin.kutter@fen-net.de wrote:

> > CGI.t seems to hang due to encoding issues - the test server (which > > emulates CGI by using open2) returns a string containing "Überall" - > > looks like it sends the content-length correctly (8 bytes), but sends > > the data in cp1252 (7 bytes). > > > > The failing test is due to SOAP::Transport::IO requiring the POSIX

> macro

> > EWOULDBLOCK, which apparently is not available on Windows. > > > > I'd suggest the following resolution: > > > > a) fix the transport or CGI test encoding issues > > b) skip the SOAP::Transport::IO test if EWOULDBLOCK is not available, > > and open a bug report for this behaviour (might take some time, and I > > think SOAP::Transport::IO is rarely used). > > > > I'd appreciate any help, as I only have limited access to a windows

> box

> > for testing. > > > > Thanks for reporting, > > > > Martin

>

Mon Jun 30 19:55:44 2008 toddr [...] cpanel.net - Correspondence added

From:

todd.e.rinaldo [...] jpmorgan.com

No output whatsoever C:\Projects\perl-soap-lite\branches\0.71>prove -v -b t\SOAP\Transport\HTTP\CGI.t t\SOAP\Transport\HTTP\CGI.... Show quoted text

> 3. ... any help appreciated :-) > I think the hanging CGI.t could be a encoding issue, where the test > script expects to read more bytes from the test server via STDIN than > this sends. I've added a alarm() call to the test, so at least it > should no longer hang > I'd be very happy if you could confirm this guess...

Mon Jul 07 11:14:59 2008 toddr [...] cpanel.net - Correspondence added

From:

todd.e.rinaldo [...] jpmorgan.com

Hi, I'm ok with turning this off for Windows. How do we do this? Is it a skip_all? If we skip all, I suggest we put an OS test in the module it's self to die if we're in windows. Thanks, Todd

Mon Dec 08 07:02:13 2008 CHORNY [...] cpan.org - Broken in 0.710.08 added

Thu Feb 12 16:13:19 2009 pshangov [...] yahoo.com - Correspondence added

What is the status of this issue? t/SOAP/Transport/HTTP/CGI sill hangs on my Straberry Perl 5.10

Mon Feb 23 02:10:54 2009 ddascalescu+perl [...] gmail.com - Correspondence added

From:

ddascalescu+perl [...] gmail.com

On Thu Feb 12 16:13:19 2009, pshangov wrote: Show quoted text

> What is the status of this issue? t/SOAP/Transport/HTTP/CGI sill hangs > on my Straberry Perl 5.10

Hangs here too, after a bunch of warnings. The last "ok" test is t/SOAP/Transport/HTTP.t. Strawberry 5.10.0.4, Windows XP Pro SP2.

Sun May 31 18:39:20 2009 CHORNY [...] cpan.org - Correspondence added

Jun 28 12:45:45 2008, martin.kutter@fen-net.de wrote: Show quoted text

> CGI.t seems to hang due to encoding issues - the test server (which > emulates CGI by using open2) returns a string containing "Überall" - > looks like it sends the content-length correctly (8 bytes), but sends > the data in cp1252 (7 bytes).

I changed data to non-Unicode string. Same result. Show quoted text

> > The failing test is due to SOAP::Transport::IO requiring the POSIX macro > EWOULDBLOCK, which apparently is not available on Windows.

Currently IO uses solution explained here: http://www.perlmonks.org/?node_id=529812 to implement non-blocking sockets on Windows. It requires to use sysread/syswrite only, but as I see SOAP::Lite already does this. After upgrade to IO 1.25 it still hangs. Interesting solution about EWOULDBLOCK is here: http://stackoverflow.com/questions/131473/ewouldblock-equivalent-errno-under-windows-perl , but it is locking-related. In Windows instead of EWOULDBLOCK, EAGAIN can be used and SOAP::Lite already does it. I added print "$_[0] $^E\n"; (into file) in sub WOULDBLOCK, and it is never called, so it is not the cause of problem. I tried to replace IPC::Open2 with IPC::Run. Debugging output that I added is showing on the screen - this means that it is not intercepted. -- Alexandr Ciornii, http://chorny.net

Sun May 31 18:49:31 2009 CHORNY [...] cpan.org - Cc CHORNY added

Tue Aug 04 15:23:24 2009 kmx [...] cpan.org - Correspondence added

Subject:

Strawberry perl 5.10 - Tests fail and then later hang [PATCH]

Hi, please find enclosed patch against SOAP-Lite-0.710.08 that makes all test PASS on Win32 strawberry 5.8.9. -- kmx

diff -u -r SOAP-Lite-0.710.08/lib/IO/SessionData.pm SOAP-Lite-0.710.08patched/lib/IO/SessionData.pm --- SOAP-Lite-0.710.08/lib/IO/SessionData.pm 2008-07-03 19:43:58.000000000 +0200 +++ SOAP-Lite-0.710.08patched/lib/IO/SessionData.pm 2009-08-04 21:09:14.421875000 +0200 @@ -30,7 +30,7 @@ ), (eval {require POSIX} ? map { - eval { POSIX->can($_)->() } + eval { POSIX->can($_) } ? (POSIX->can($_)->() => 1) : () } @names diff -u -r SOAP-Lite-0.710.08/lib/SOAP/Transport/HTTP.pm SOAP-Lite-0.710.08patched/lib/SOAP/Transport/HTTP.pm --- SOAP-Lite-0.710.08/lib/SOAP/Transport/HTTP.pm 2008-07-13 22:30:54.000000000 +0200 +++ SOAP-Lite-0.710.08patched/lib/SOAP/Transport/HTTP.pm 2009-08-04 21:19:57.515625000 +0200 @@ -495,8 +495,10 @@ my $content = q{}; my $buffer; binmode(STDIN); - while (read(STDIN,$buffer,$length)) { + ###NOTE: using some non-blocking IO + select() would be better + while (sysread(STDIN,$buffer,$length)) { $content .= $buffer; + last if(length($content)>=$length); } $self->request(HTTP::Request->new(

Tue Sep 29 16:38:08 2009 kutterma [...] users.sourceforge.net - Correspondence added

Hi, fixed as #49011: Fails to install under strawberry perl Thanks ! Martin

Tue Sep 29 16:38:09 2009 kutterma [...] users.sourceforge.net - Status changed from 'open' to 'resolved'