Bug #85713 for SOAP-Lite: Bug in SOAP::Transport::HTTP, 500 error skips parsing of response for 0.716

Wed May 29 18:02:22 2013 danielb [...] libentech.com - Ticket created

Subject:	Bug in SOAP::Transport::HTTP, 500 error skips parsing of response for 0.716
Date:	Wed, 29 May 2013 17:02:01 -0500
To:	bug-SOAP-Lite [...] rt.cpan.org
From:	Daniel Baber <danielb [...] libentech.com>

When I make a SOAP call to a service that returns a 500 Internal Server Error with a SOAP fault in the response body the response is not parsed and is undef. I believe this is due to recent change in 0.716 in SOAP::Transport::HTTP as shown is the diff between 0.715 and 0.716: @@ -303,6 +304,7 @@ sub send_receive { $self->message( $self->http_response->message ); $self->is_success( $self->http_response->is_success ); $self->status( $self->http_response->status_line ); + return if ($self->http_response->is_success == 0); So for a 500 error is_success is equal to 0. When I rolled things back to 0.715 then my unit tests that checked for various error conditions in my SOAP call(s) started working again. -- "I sincerely believe ... that banking establishments are more dangerous than standing armies, and that the principle of spending money to be paid by posterity, under the name of funding, is but swindling futurity on a large scale." -- Thomas Jefferson in a letter to John Taylor May 28, 1816 Support The Campaign For Liberty - http://www.campaignforliberty.com, Sign up Now!

Wed May 29 19:51:48 2013 fred [...] redhotpenguin.com - Correspondence added

Subject:	Re: [rt.cpan.org #85713] Bug in SOAP::Transport::HTTP, 500 error skips parsing of response for 0.716
Date:	Wed, 29 May 2013 16:51:29 -0700
To:	bug-SOAP-Lite [...] rt.cpan.org
From:	Fred Moyer <fred [...] redhotpenguin.com>

Thanks for the report. Any chance you have a short unit test to contribute that exercises this? On May 29, 2013 3:02 PM, "Daniel Baber via RT" <bug-SOAP-Lite@rt.cpan.org> wrote: Show quoted text

> Wed May 29 18:02:22 2013: Request 85713 was acted upon. > Transaction: Ticket created by danielb@libentech.com > Queue: SOAP-Lite > Subject: Bug in SOAP::Transport::HTTP, 500 error skips parsing of > response for 0.716 > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: danielb@libentech.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=85713 > > > > When I make a SOAP call to a service that returns a 500 Internal Server > Error with a SOAP fault in the response body the response is not parsed and > is undef. I believe this is due to recent change in 0.716 in > SOAP::Transport::HTTP as shown is the diff between 0.715 and 0.716: > > @@ -303,6 +304,7 @@ sub send_receive { > $self->message( $self->http_response->message ); > $self->is_success( $self->http_response->is_success ); > $self->status( $self->http_response->status_line ); > + return if ($self->http_response->is_success == 0); > > > So for a 500 error is_success is equal to 0. When I rolled things back to > 0.715 then my unit tests that checked for various error conditions in my > SOAP call(s) started working again. > -- > "I sincerely believe ... that banking establishments are more dangerous > than standing armies, and that the principle of spending money to be paid > by posterity, under the name of funding, is but swindling futurity on a > large scale." > -- Thomas Jefferson in a letter to John Taylor May 28, 1816 > > Support The Campaign For Liberty - http://www.campaignforliberty.com, Sign > up Now! > >

Wed May 29 19:51:48 2013 The RT System itself - Status changed from 'new' to 'open'

Wed Jun 26 02:35:05 2013 PHRED [...] cpan.org - Status changed from 'open' to 'resolved'