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

Jonathan Moore commented on HTTPCLIENT-981:
-------------------------------------------

As others have said, this check originates from the original design of the 
caching module as a decorator around a backend HttpClient; modelling the 
module's behavior as a proxy seemed like a good model for understanding the 
protocol requirements for it. 

That said, the caching module isn't *actually* a proxy--it's just *part* of a 
client. As such, some of the proxy requirements can end up a little awkward. 
This is probably one of those cases; especially since the DefaultHttpClient 
already appropriately computes Content-Length, I'd be ok with removing this 
check from the caching module.


> CachingHttpClient returns a 411 respones when executing a POST (HttpPost) 
> request 
> ----------------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-981
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-981
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: Cache
>    Affects Versions: 4.1 Alpha2
>            Reporter: Vianney Carel
>             Fix For: 4.1 Alpha3
>
>
> The CachingHttpClient validates requests prior executing them, by calling 
> RequestProtocolCompliance.requestIsFatallyNonCompliant(..).
> When executing an HttpPost, this method considers the request is invalid 
> because it does not contain (yet) a content-length header. Indeed, I observed 
> that this header is generated at the time the DefaultHttpClient fires the 
> request.
> NB: i'm using the Cache 4.1-alpha2 plugged over the HttpClient 4.0.1-final. I 
> can't use the latest version for both because I need to rely on a stable 
> version if there's any. I would be curious to know if we get the same 
> behaviour in 4.1...
> Anyway, I would see two fixes for that issue:
> - make HttpPost set the content-length at the time the entity is set,
> - or remove the validation step on the CachingHttpClient side.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org
For additional commands, e-mail: dev-h...@hc.apache.org

Reply via email to