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


Thanks,

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

Reply via email to