hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HTTPCLIENT-1865) DefaultServiceUnavailableRetryStrategy does not respect HttpEntity#isRepeatable
Date Tue, 01 Aug 2017 17:29:00 GMT

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

ASF subversion and git services commented on HTTPCLIENT-1865:

Commit 9efcba87302dc76fe31b33d685bca1fea34cf7f2 in httpcomponents-client's branch refs/heads/master
from [~tjcelaya]
[ https://git-wip-us.apache.org/repos/asf?p=httpcomponents-client.git;h=9efcba8 ]

[HTTPCLIENT-1865] DefaultServiceUnavailableRetryStrategy does not
respect HttpEntity#isRepeatable.

> DefaultServiceUnavailableRetryStrategy does not respect HttpEntity#isRepeatable
> -------------------------------------------------------------------------------
>                 Key: HTTPCLIENT-1865
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1865
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpClient (classic)
>    Affects Versions: 4.5.3
>         Environment: Any environment
>            Reporter: Tomas Celaya
>             Fix For: 4.5.4
>         Attachments: 0001-Only-retry-request-when-HttpEntity.isRepeatable-retu.patch,
> The {{HttpEntity}} interface provides a method for indicating whether or not an entity
can be automatically retried. While investigating an issue with corruption of encrypted uploads
in the [java-manta library|https://github.com/joyent/java-manta] I discovered that requests
were being automatically retried in the event of a 503 response from the server. Stepping
through the execution path led me to discover that {{ServiceUnavailableRetryExec}} does not
check the supplied {{HttpRequest}} to determine whether or not it can be safely retried. We've
implemented a fix within java-manta in our subcclass of {{DefaultServiceUnavailableRetryStrategy}}
but this seems like it could bite others as well so I thought it would merit a bug report.
> The included patch corrects this behavior. Checked that {{mvn verify}} completes successfully.
Please let me know if there are any questions or other modifications that I need to make in
order to contribute this patch.

This message was sent by Atlassian JIRA

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

View raw message