You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hc.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2017/08/01 01:11:00 UTC
[jira] [Commented] (HTTPCLIENT-1865)
DefaultServiceUnavailableRetryStrategy does not respect
HttpEntity#isRepeatable
[ https://issues.apache.org/jira/browse/HTTPCLIENT-1865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16108239#comment-16108239 ]
ASF subversion and git services commented on HTTPCLIENT-1865:
-------------------------------------------------------------
Commit 7dfd236dc155e97c650c9da9b4b3f9234b3b2de9 in httpcomponents-client's branch refs/heads/4.5.x from [~tjcelaya]
[ https://git-wip-us.apache.org/repos/asf?p=httpcomponents-client.git;h=7dfd236 ]
[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
> Attachments: 0001-Only-retry-request-when-HttpEntity.isRepeatable-retu.patch, 0002-ServiceUnavailableRetryExec-test-case-for-verifying-.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
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org