michael-o commented on issue #57: WAGON-567: support retry on server side errors URL: https://github.com/apache/maven-wagon/pull/57#issuecomment-544947921 @ColinHebert I really much like your proposal. As soon as this goes in I will deprecate and mark the current backoff mechanism for removal in 4.0.0. I am considering to modify `DefaultServiceUnavailableRetryStrategy` by accepting a set of possible status codes. I will discuss this with other HttpComponents committers since I am one. I think that the current backoff mechanism will be unavailable as soon as a new handler pops in or it will fail and the mech would pop in. My proposal is: * Lower default retry to 3 because the low level retry handler uses 3 too * Raise from default 1000 ms to 2000 ms to 2500 ms not to bomb a server and have a closer value to `maven.wagon.httpconnectionManager.backoffSeconds` * Introduce a Maven-default ("default") which handles 503 and 429. * Rename the custom impl to "extended" because it covers way more status codes. WDY?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
