[ 
http://issues.apache.org/jira/browse/AXIS2-824?page=comments#action_12416855 ] 

Chuck Williams commented on AXIS2-824:
--------------------------------------

Oleg,

Sorry to take so long to look at this -- on the road.

HTTPWorker only sets the version and chunking settings if they have been 
explicitly speicifed in the axis2 configuration.  It seems to me that such an 
explicit configuration should take precendence over the automatic assignments 
made by HttpCore.

Do you see a material problem with the code as is, or is this proposed change 
just a theoretical concern?

Chuck


> HTTPWorker be should not be unecessarily modifying transport settings such as 
> HTTP version and content chunking
> ---------------------------------------------------------------------------------------------------------------
>
>          Key: AXIS2-824
>          URL: http://issues.apache.org/jira/browse/AXIS2-824
>      Project: Apache Axis 2.0 (Axis2)
>         Type: Improvement

>   Components: transports
>     Versions: 1.0
>     Reporter: Oleg Kalnichevski
>  Attachments: simplehttpserver-cleanup.patch
>
> Presently HTTPWorker class unnecessarily meddles with HTTP transport 
> parameters such content chunking and HTTP protocol version. In most cases 
> HttpCore is much better equipped to pick up optimal transport settings 
> depending on properties of the incoming request. HttpCore automatically 
> downgrades the protocol version to HTTP/1.0 if the client does not support 
> HTTP/1.1. Likewise it automatically applies the chunk coding if the content 
> length is unknown.
> I understand this one may be a little controversial, nonetheless I would like 
> to put forth this patch for your consideration
> Oleg

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
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