[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-10-13 Thread Ned Deily
Changes by Ned Deily : -- nosy: +orsenthil ___ Python tracker ___ ___ Python-bugs-list

[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-10-13 Thread R. David Murray
Changes by R. David Murray : -- stage: patch review -> needs patch ___ Python tracker ___

[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-10-13 Thread R. David Murray
R. David Murray added the comment: Well, it's missing doc changes and tests, so even if it is still applicable it isn't ready for merge yet. -- ___ Python tracker

[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-10-13 Thread Stéphane Wirtel
Changes by Stéphane Wirtel : -- stage: -> patch review ___ Python tracker ___ ___

[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-10-13 Thread orban
orban added the comment: Just checked (for my first contribution at cpython) this patch. For me this patch could be merge. -- nosy: +matorban ___ Python tracker

[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-09-27 Thread secynic
Changes by secynic : -- keywords: +patch versions: -Python 2.7, Python 3.3, Python 3.4, Python 3.5, Python 3.6 Added file: http://bugs.python.org/file44850/issue28291.patch ___ Python tracker

[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-09-27 Thread secynic
secynic added the comment: It is a very limited use case; I won't gripe about 3.7+ only support. At the end of the day, even shown in the comments of that class, it shouldn't be set to a static 5 count. At least this is a very easy patch that won't affect existing code. --

[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-09-27 Thread R. David Murray
R. David Murray added the comment: For (convoluted) background, see issue 8797. A solution that uses a new would be a new feature and could only go in 3.7, but perhaps the solutions in the above issue will point to a backward compatible solution. -- nosy: +r.david.murray

[issue28291] urllib/urllib2 AbstractDigestAuthHandler locked to retried count of 5

2016-09-27 Thread secynic
New submission from secynic: urllib/urllib2 AbstractDigestAuthHandler is hardcoded to 5 retries (self.retried). Normally this wouldn't be an issue. Certain products link basic HTTP auth to Active Directory (yes, this shouldn't be a thing). When you have a failed login attempt lockout set on