[ 
https://issues.apache.org/jira/browse/HTTPCLIENT-1073?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13009820#comment-13009820
 ] 

Jon Moore commented on HTTPCLIENT-1073:
---------------------------------------

I believe the correct fix for this problem is one of:
(a) document the proper workaround for the client to avoid the connection leak 
(perhaps using HttpUriRequest#abort)
(b) have the caching module consume the response body if present before 
throwing the exception
(c) don't throw an exception here and just treat the noncompliant response as 
cache-invalidating and uncacheable.

I think my personal preference is (c) and will start working on a patch to that 
effect.


> cache module generates exceptions for non-compliant responses without 
> consuming response bodies
> -----------------------------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-1073
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1073
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: Cache
>    Affects Versions: 4.1 Beta1, 4.1 Final, 4.1.1, 4.2 Final
>            Reporter: Jon Moore
>            Assignee: Jon Moore
>
> In the ResponseProtocolCompliance class, the caching module checks the 
> incoming origin response to attempt to make it compliant with RFC2616. 
> However, if there are instances where this is not possible, it currently 
> throws an exception without consuming the origin response body; this causes a 
> connection leak if the general try..catch..finally pattern documented on the 
> HttpClient interface Javadoc is followed.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to