Re: [Bug-wget] [PATCH] Add support for --retry-http503

[Tom Szilagyi]

On Tue, Jan 31, 2017 at 11:24:33AM +0100, Giuseppe Scrivano wrote:
> Hi Tom,
> 
> Tom Szilagyi <address@hidden> writes:
> 
> > Consider HTTP 503 (Service unavailable) as a non-fatal, transient
> > error. Normally Wget gives up immediately on receiving this HTTP
> > response. Certain special use cases might require Wget to retry even
> > in the face of this error. With this option, such retries are
> > performed subject to the normal retry timing and retry count
> > limitations of Wget. Using this option is generally not recommended.
> 
> thanks for working on it.  If we are going to add something like this, I
> think it should be made more generic.  What about something like:
> 
> --retry-on-http-error=503,504,505...

Hi Giuseppe,

Sure, that is entirely doable from a technical point of view.

However, I have thought about this beforehand, and to me, 503 and 504
were the only 5xx errors potentially worth retrying. All the others
signal fatal conditions. And Wget already retries 504 without any
patch. Hence, the patch is for 503 only.

Let me know if you still want me to prepare a new patch based on your
proposal.

Thanks,
Tom