[issue8797] urllib2 basicauth broken in 2.6.5: RuntimeError: maximum recursion depth exceeded in cmp

[Senthil Kumaran]

Senthil Kumaran <orsenthil at gmail.com> added the comment:

On Thu, Aug 26, 2010 at 3:24 PM, Mads Kiilerich <report at bugs.python.org> wrote:
> Mads Kiilerich <mads at kiilerich.com> added the comment:
>
> Senthil, can you tell us why this fix is correct - and convince us that it is the Final Fix for this issue?

Hello Mads, this may not be be final fix. I just adopted what was
already done for Digest Auth for the Basic Auth, it could be an
interim solution that avoids recursion problem.

I agree with you respect to the other error codes, there is already
another bug open to handle this. The reset counter is reset on success
too, in another part of the code. I shall respond to the other
questions a bit later (1 week, I am kind of busy at the moment) and
lets go for the best fix as discussed.

Thanks,
Senthil

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue8797>
_______________________________________