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

ASF subversion and git services commented on HTTPCLIENT-2366:
-------------------------------------------------------------

Commit 2f57d441dc92aa0e087a47d7bb65c3263f85d0fd in httpcomponents-client's 
branch refs/heads/5.4.x from Arturo Bernal
[ https://gitbox.apache.org/repos/asf?p=httpcomponents-client.git;h=2f57d441d ]

HTTPCLIENT-2366 - Fix isRepeatable method in RequestEntityProxy to return true 
before consumption. (#630)



> HttpClient5 - RequestEntityProxy#isRepeatable incorrectly returns false 
> before entity is consumed
> -------------------------------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-2366
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-2366
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpClient (classic)
>            Reporter: Nicklas Wallgren
>            Priority: Major
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> The behavior of {{RequestEntityProxy#isRepeatable}} changed in [this 
> commit|https://github.com/apache/httpcomponents-client/commit/a1eed8084de7c58203a6acb799b7e4b04f669d39#r154843658],
>  where the method now returns {{false}} unless {{writeTo()}} has already been 
> called.
> This appears to be a regression. Previously, an entity was considered 
> repeatable as long as it hadn't been consumed yet and the original entity 
> itself was repeatable. The new logic incorrectly assumes that an entity is 
> only repeatable _after_ it's been consumed, which defeats the purpose of 
> determining retry eligibility _before_ transmission.



--
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