[ 
https://issues.apache.org/jira/browse/GERONIMO-3839?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12568046#action_12568046
 ] 

Rick McGuire commented on GERONIMO-3839:
----------------------------------------

It seems like HttpRequestEncoder should be overriding those particular headers 
rather than not adding them.  I agree there should be a single set, but it 
seems like the ones added by the HttpRequestEncoder should be consider the 
"correct" set. 

> some request headers may be added twice
> ---------------------------------------
>
>                 Key: GERONIMO-3839
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-3839
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: AsyncHttpClient
>    Affects Versions: 1.x
>            Reporter: Sangjin Lee
>            Assignee: Rick McGuire
>            Priority: Minor
>
> Some request headers get special treatment by HttpRequestEncoder.  
> HttpRequestEncoder does not make any effort in checking to see if they are 
> present in the headers already.  As a result, they may be added twice if one 
> is not careful.  For example,
> - Content-Type & Content-Length are added by HttpRequestEncoder for POST 
> requests, and should not be added by callers.
> - Host & User-Agent are always added by HttpRequestEncoder, and should not be 
> added by callers as ordinary headers.
> Although one could argue that callers should not add these headers by hand, I 
> think HttpRequestEncoder still should ensure that they are not added twice.

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

Reply via email to