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

Julio J. Gomez Diaz commented on HTTPCORE-781:
----------------------------------------------

Thanks for your time and your kind answer [~olegk] IMHO this was not a bogus 
bug because the RFC does not mandate any specific transfer encoding (not gzip) 
but DOES show that a comma separated value with more than 1 value MUST be 
supported. You have centered your answer in the "gzip" value, but I'm not 
centered in that specific value (nor the RFC), it says that a comma separated 
value with more than 1 item is VALID.

I understand that you are backing already discussed design decisions (maybe 
right, maybe wrong); but IMHO you should understand that, if they are not 
documented, those changes can appear to be a legitimate bug to other developers 
(like me).

> DefaultContentLengthStrategy does not conform to RFC 9112 regarding 
> Transfer-encoding HTTP header
> -------------------------------------------------------------------------------------------------
>
>                 Key: HTTPCORE-781
>                 URL: https://issues.apache.org/jira/browse/HTTPCORE-781
>             Project: HttpComponents HttpCore
>          Issue Type: Bug
>          Components: HttpCore
>    Affects Versions: 5.3.3
>            Reporter: Julio J. Gomez Diaz
>            Priority: Major
>
> I noticed that changes for 5.3.x in class 
> "org.apache.hc.core5.http.impl.DefaultContentLengthStrategy" (this commit: 
> [https://github.com/apache/httpcomponents-core/commit/7fea9ef7fa2be53a7631aad9bb484db7347667be#diff-385ee34bb5e36b01eb7c355de9557d7f830ba5a202f156057ac3559507ea5c7aR44-R68]
>  ) are not compliant to what is described in RFC 9112 section 6.1 ( 
> [https://www.rfc-editor.org/rfc/rfc9112.html#section-6.1] ) at least in this 
> example, presented in the RFC section as a VALID example: 
> {code:java}
> Transfer-Encoding: gzip, chunked{code}
> With 5.3.x it fails throwing an exception via this line:
> {code:java}
>  throw new NotImplementedException("Unsupported transfer encoding: " + 
> teh.getValue());{code}
> effectively printing in the stack trace:
> "Unsupported transfer encoding: gzip,chunked"
>  
>  
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to