You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Ryan Lubke <Ry...@Sun.COM> on 2002/09/05 05:50:00 UTC

[HttpClient][PATCH] Fix for NPE in PutMethod due to new StatusLine changes

Had to add an additional check to make sure StatusLine returned from
getStatusLine was not null before attempting to access the status-code.

Unit tests using the PutMethod now pass.

Also a minor update to the log output within HttpMethodBase.



Re: [HttpClient][PATCH] Fix for NPE in PutMethod due to new StatusLine changes

Posted by Ryan Lubke <Ry...@Sun.COM>.
dIon,

Please look at my second post on this subject.  That is the patch that
should be applied.  I made a mistake with the first one.



On Thu, 2002-09-05 at 00:27, dion@multitask.com.au wrote:
> Applied,
> 
> thanks... I still have some local test errors, but I think they're due to 
> out of date code. I'm cleaning and retesting.
> 
> --
> dIon Gillard, Multitask Consulting
> Work:      http://www.multitask.com.au
> Developers: http://adslgateway.multitask.com.au/developers
> 
> 
> Ryan Lubke <Ry...@Sun.COM> wrote on 05/09/2002 01:50:00 PM:
> 
> > Had to add an additional check to make sure StatusLine returned from
> > getStatusLine was not null before attempting to access the status-code.
> > 
> > Unit tests using the PutMethod now pass.
> > 
> > Also a minor update to the log output within HttpMethodBase.
> > 
> > 
> > [attachment "patch.txt" removed by dIon Gillard/Multitask Consulting/AU] 
> --
> > To unsubscribe, e-mail: 
> <ma...@jakarta.apache.org>
> > For additional commands, e-mail: 
> <ma...@jakarta.apache.org>
> 
> --
> To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
> For additional commands, e-mail: <ma...@jakarta.apache.org>
> 



--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


Re: [HttpClient][PATCH] Fix for NPE in PutMethod due to new StatusLine changes

Posted by di...@multitask.com.au.
Applied,

thanks... I still have some local test errors, but I think they're due to 
out of date code. I'm cleaning and retesting.

--
dIon Gillard, Multitask Consulting
Work:      http://www.multitask.com.au
Developers: http://adslgateway.multitask.com.au/developers


Ryan Lubke <Ry...@Sun.COM> wrote on 05/09/2002 01:50:00 PM:

> Had to add an additional check to make sure StatusLine returned from
> getStatusLine was not null before attempting to access the status-code.
> 
> Unit tests using the PutMethod now pass.
> 
> Also a minor update to the log output within HttpMethodBase.
> 
> 
> [attachment "patch.txt" removed by dIon Gillard/Multitask Consulting/AU] 
--
> To unsubscribe, e-mail: 
<ma...@jakarta.apache.org>
> For additional commands, e-mail: 
<ma...@jakarta.apache.org>

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>