Closed
Bug 676846
Opened 10 years ago
Closed 10 years ago
[about:permissions] Basic Auth Passwords not taken into Account
Categories
(Firefox :: Preferences, defect)
Firefox
Preferences
Tracking
()
RESOLVED
FIXED
Firefox 9
People
(Reporter: seymor42, Assigned: felix)
References
Details
Attachments
(2 files)
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:8.0a1) Gecko/20110805 Firefox/8.0a1 Build ID: 20110805030744 Steps to reproduce: Went to about:permissions, typed a domain for which I had Basic Auth credentials saved into the 'Search Sites' box. Actual results: Under the Store Passwords section, it says '0 passwords are stored for this web site.' and the Manage Passwords... button is disabled. Expected results: Should display how many passwords I have, and enable the Manage Passwords... button. I can clear the 'Search Sites' box (so I am just looking at All Sites) and click on the Manage Passwords... button. Then search for the domain that I have the saved Basic Auth credentials, and it shows my saved passwords for that domain.
Reporter | ||
Updated•10 years ago
|
Component: General → Preferences
![]() |
||
Comment 1•10 years ago
|
||
Confirmed against Mozilla/5.0 (Windows NT 5.1; rv:8.0a1) Gecko/20110807 Firefox/8.0a1 ID:20110807030727 PS. Site Visits do work.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 7 → All
Hardware: x86_64 → All
Summary: about:permissions basic auth 0 passwords are stored for this web site → [about:permissions] Basic Auth Passwords not taken into Account
Version: 8 Branch → Trunk
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → ffung
Assignee | ||
Comment 2•10 years ago
|
||
Summary of Changes - Changed last parameter of findLogins from null to "". When it's null, it won't match anything with a login scheme (e.g. Mozilla LDAP) where as "" will freely match against anything.
Attachment #560474 -
Flags: review?(margaret.leibovic)
Updated•10 years ago
|
Attachment #560474 -
Flags: review?(margaret.leibovic) → review?(dolske)
Comment 3•10 years ago
|
||
Comment on attachment 560474 [details] [diff] [review] findLogins should ignore httpScheme This makes sense to me, but Dolske would better understand the implications of this change, since he's more familiar with the login manager code. I also think you should leave that comment in there, since it still won't find logins that could be stored with different URIs from that host.
Attachment #560474 -
Flags: feedback+
Comment 4•10 years ago
|
||
Comment on attachment 560474 [details] [diff] [review] findLogins should ignore httpScheme Seems fine to me.
Attachment #560474 -
Flags: review?(dolske) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/184cfbecb1c3
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
You need to log in
before you can comment on or make changes to this bug.
Description
•