Patch #812285: allow multiple auth schemes in AbstractBasicAuthHandler.
(backport from rev. 54195)
This commit is contained in:
parent
6a9c780cde
commit
fd13ef9c9c
|
@ -766,11 +766,10 @@ class HTTPPasswordMgrWithDefaultRealm(HTTPPasswordMgr):
|
|||
|
||||
class AbstractBasicAuthHandler:
|
||||
|
||||
rx = re.compile('[ \t]*([^ \t]+)[ \t]+realm="([^"]*)"', re.I)
|
||||
# XXX this allows for multiple auth-schemes, but will stupidly pick
|
||||
# the last one with a realm specified.
|
||||
|
||||
# XXX there can actually be multiple auth-schemes in a
|
||||
# www-authenticate header. should probably be a lot more careful
|
||||
# in parsing them to extract multiple alternatives
|
||||
rx = re.compile('(?:.*,)*[ \t]*([^ \t]+)[ \t]+realm="([^"]*)"', re.I)
|
||||
|
||||
# XXX could pre-emptively send auth info already accepted (RFC 2617,
|
||||
# end of section 2, and section 1.2 immediately after "credentials"
|
||||
|
|
Loading…
Reference in New Issue