Closed
Bug 239734
Opened 20 years ago
Closed 20 years ago
negotiateauth extension should not request mutual auth
Categories
(Core :: Networking, defect)
Core
Networking
Tracking
()
RESOLVED
FIXED
mozilla1.7final
People
(Reporter: darin.moz, Assigned: darin.moz)
Details
(Keywords: helpwanted)
Attachments
(1 file)
|
735 bytes,
patch
|
cneberg
:
review+
darin.moz
:
superreview+
mkaply
:
approval1.7+
|
Details | Diff | Splinter Review |
per these comments: see bug 17578 comment #241 see bug 17578 comment #242 see bug 17578 comment #243 the negotiateauth extension currently requests mutual auth but does not implement mutual auth. we should not request mutual auth unless we are going to fully implement it.
|
Assignee | |
Comment 1•20 years ago
|
||
we should try to fix this for 1.7 final if possible. anyone want to help by contributing a patch?
|
||
Comment 2•20 years ago
|
||
Just set 'req_flags = 0;', simple enough, I think...
|
|
Assignee | |
Comment 3•20 years ago
|
||
Comment on attachment 145503 [details] [diff] [review] simple fix... makes sense to me. sr=me provided folks agree that this is correct.
Attachment #145503 -
Flags: superreview+
Attachment #145503 -
Flags: review?(cneberg)
|
||
Comment 4•20 years ago
|
||
Comment on attachment 145503 [details] [diff] [review] simple fix... Looks good.
Attachment #145503 -
Flags: review?(cneberg) → review+
|
|
Assignee | |
Comment 5•20 years ago
|
||
Comment on attachment 145503 [details] [diff] [review] simple fix... this is a low-risk patch for a low-use feature ;-)
Attachment #145503 -
Flags: approval1.7?
|
||
Comment 6•20 years ago
|
||
Comment on attachment 145503 [details] [diff] [review] simple fix... a=mkaply
Attachment #145503 -
Flags: approval1.7? → approval1.7+
|
|
Assignee | |
Comment 7•20 years ago
|
||
fixed-on-trunk for 1.7 final
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•