[email protected] wrote:

> This one is so small that I'm attaching the diffs instead of generating
> a webrev.
> 
> A user recently hit a problem with a gzip encoded download that kept
> failing with an E_BAD_CONTENT_ENCODING error.  It turns out that if the
> gzip stream gets a CRC error, we can return this error.  It should
> be retryable, in case the corruption is transient.
> 
> diff -r c9985876d799 -r 4f72231b5108 src/modules/client/transport/exception.py
> --- a/src/modules/client/transport/exception.py Tue Nov 09 13:27:02 2010 -0800
> +++ b/src/modules/client/transport/exception.py Tue Nov 09 14:16:15 2010 -0800
> @@ -65,7 +65,7 @@
>  retryable_pycurl_errors = set((pycurl.E_COULDNT_CONNECT, 
> pycurl.E_PARTIAL_FILE,
>      pycurl.E_OPERATION_TIMEOUTED, pycurl.E_GOT_NOTHING, pycurl.E_SEND_ERROR,
>      pycurl.E_RECV_ERROR, pycurl.E_COULDNT_RESOLVE_HOST,
> -    pycurl.E_TOO_MANY_REDIRECTS))
> +    pycurl.E_TOO_MANY_REDIRECTS, pycurl.E_BAD_CONTENT_ENCODING))
>  
>  class TransportException(api_errors.TransportError):
>          """Base class for various exceptions thrown by code in transport

LGTM.

Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to