Bug 93386

Summary: [BlackBerry] Regression in credential storage when using Negotiate auth
Product: WebKit Reporter: Joe Mason <joenotcharles>
Component: Page LoadingAssignee: Joe Mason <joenotcharles>
Severity: Normal CC: mifenton, webkit.review.bot
Priority: P1    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
fix with updated changelog none

Description Joe Mason 2012-08-07 13:12:46 PDT
r124205 regressed credential storage and lookup.  It added code to update the scheme used in credential storage if the scheme chosen by the network stack is different from that requested by webkit (currently this is only used when webkit requests Negotiate auth, but the Negotiate infrastructure isn't set up correctly, so the network stack falls back to another auth type, usually NTLM).  But the credentials are saved when a "success" status is received, and the scheme in the credentials is not updated until notifyAuthReceived is called to report the actual auth scheme used.  The result is that the credentials are stored with the wrong auth scheme and can never be retreived.
Comment 1 Joe Mason 2012-08-07 14:04:22 PDT
Created attachment 156995 [details]
Comment 2 Joe Mason 2012-08-07 14:06:07 PDT
Comment on attachment 156995 [details]

Forgot to put this up for internal review first.
Comment 3 Joe Mason 2012-08-07 14:14:13 PDT
Created attachment 156999 [details]
fix with updated changelog
Comment 4 WebKit Review Bot 2012-08-07 21:34:41 PDT
Comment on attachment 156999 [details]
fix with updated changelog

Clearing flags on attachment: 156999

Committed r124980: <http://trac.webkit.org/changeset/124980>
Comment 5 WebKit Review Bot 2012-08-07 21:34:44 PDT
All reviewed patches have been landed.  Closing bug.