| Subject: | Re: Bug in SOAP::Transport::HTTP, 500 error skips parsing of response for 0.716 |
| Date: | Wed, 29 May 2013 19:34:03 -0500 |
| To: | "bug-SOAP-Lite [...] rt.cpan.org" <bug-SOAP-Lite [...] rt.cpan.org> |
| From: | Daniel Baber <danielb [...] libentech.com> |
I can probably put something together but I'd have to mock the SOAP call,
wouldn't I?
On Wednesday, May 29, 2013, fred@redhotpenguin.com via RT wrote:
Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=85713 >
>
> 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<javascript:;>
--
"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!
> >
> wrote:
>
> > Wed May 29 18:02:22 2013: Request 85713 was acted upon.
> > Transaction: Ticket created by danielb@libentech.com <javascript:;>
> > 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 <javascript:;>
> > 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!
> >
> >
>
>