Rev 2543: Take Aaron's comments into account. in file:///v/home/vila/src/bugs/120678/

[Vincent Ladeuil]

At file:///v/home/vila/src/bugs/120678/

------------------------------------------------------------
revno: 2543
revision-id: v.ladeuil+lp at free.fr-20070703152901-o25i9ntsv1jj1m4h
parent: v.ladeuil+lp at free.fr-20070626091244-nou3usgr34947wz9
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: 120678
timestamp: Tue 2007-07-03 17:29:01 +0200
message:
  Take Aaron's comments into account.
modified:
  bzrlib/transport/http/_urllib2_wrappers.py _urllib2_wrappers.py-20060913231729-ha9ugi48ktx481ao-1
-------------- next part --------------
=== modified file 'bzrlib/transport/http/_urllib2_wrappers.py'
--- a/bzrlib/transport/http/_urllib2_wrappers.py	2007-06-26 09:12:44 +0000
+++ b/bzrlib/transport/http/_urllib2_wrappers.py	2007-07-03 15:29:01 +0000
@@ -728,7 +728,7 @@
     Note: the proxy handling *may* modify the protocol used; the request may be
     against an https server proxied through an http proxy. So, https_request
     will be called, but later it's really http_open that will be called. This
-    explain why we don't have to call self.parent.open as the urllib2 did.
+    explains why we don't have to call self.parent.open as the urllib2 did.
     """
 
     # Proxies must be in front
@@ -880,7 +880,7 @@
       successful and the request authentication parameters have been updated.
     """
 
-    _max_retry = 3
+    _max_retry = 2
     """We don't want to retry authenticating endlessly"""
 
     # The following attributes should be defined by daughter
@@ -911,7 +911,7 @@
         # Don't try  to authenticate endlessly
         if self._retry_count is None:
             # The retry being recusrsive calls, None identify the first try
-            self._retry_count = 1
+            self._retry_count = 0
         else:
             self._retry_count += 1
             if self._retry_count > self._max_retry: