Remembering credentials while the "Saved Passwords" window is open can cause temporary a wrong ordering

RESOLVED DUPLICATE of bug 211352

Status

()

RESOLVED DUPLICATE of bug 211352
3 years ago
2 years ago

People

(Reporter: sworddragon2, Unassigned)

Tracking

40 Branch
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [passwords:management])

(Reporter)

Description

3 years ago
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:40.0) Gecko/20100101 Firefox/40.0
Build ID: 20150820085905



Actual results:

After going to the "Saved Passwords" window to copy the username/password from there into a https version of the related site and remembering the new credentials I have noticed that the ordering in the "Saved Passwords" window broke until it got opened again.


Expected results:

Remembering new credentials while the "Saved Passwords" window is open should correctly sort it.
Hello,

I understand exactly what you are saying here but could you provide me with exact links to sites so that I can see exactly what you are seeing?

Thank you,

Justin
Flags: needinfo?(sworddragon2)
(Reporter)

Comment 2

3 years ago
This should work with any site that provides a http and https version.
Flags: needinfo?(sworddragon2)

Comment 3

3 years ago
I don't understand this bug, can you explain it?
Flags: needinfo?(sworddragon2)
(Reporter)

Comment 4

3 years ago
In about:preferences#security open the "Saved Logins" dialog and while it is open migrate manually a login from http to https by visiting a website. After selecting to remember your credentials the ordering of the still opened "Saved Logins" dialog temporary break.
Flags: needinfo?(sworddragon2)
What do you mean by (In reply to sworddragon2 from comment #4)
> After selecting to remember your credentials the ordering of the still
> opened "Saved Logins" dialog temporary break.

The new password was and user name combination was added and the dialog was still functional.
Flags: needinfo?(sworddragon2)
(Reporter)

Comment 6

3 years ago
Steps to reproduce:

1. Start Firefox and go to about:preferences#security.
2. Click on "Saved Logins..." and sort by "Username" (ascending).
3. Search for "https://bugzilla.mozilla.org".
4. Go to https://bugzilla.mozilla.org.
5. Login as testb with any password.
6. Click on "Remember" to store the credentials.
7. Login as testa with any password.
8. Click on "Remember" to store the credentials.


Actual results:

testb is sorted before testa.


Expected results:

testb should be sorted after testa.
Flags: needinfo?(sworddragon2)
When adding a new password it sorts by url. (In reply to sworddragon2 from comment #6)
> Steps to reproduce:
> 
> 1. Start Firefox and go to about:preferences#security.
> 2. Click on "Saved Logins..." and sort by "Username" (ascending).
> 3. Search for "https://bugzilla.mozilla.org".
> 4. Go to https://bugzilla.mozilla.org.
> 5. Login as testb with any password.
> 6. Click on "Remember" to store the credentials.
> 7. Login as testa with any password.
> 8. Click on "Remember" to store the credentials.
> 
> 
> Actual results:
> 
> testb is sorted before testa.
> 
> 
> Expected results:
> 
> testb should be sorted after testa.

Since testb was added first, it has precedence over testa. You can how ever manually sort by username which would then appropriately sort it to your liking.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → INVALID
(Reporter)

Comment 8

3 years ago
(In reply to Justin - QA from comment #7)
> When adding a new password it sorts by url.

This is not the case here. Adding a new password still does sort by Username as previously selected. This also means that the sorting is wrong with testb being before testa.
Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
I can not reproduce this issue as my sort for username places testa before testb when selecting sort by username before adding test a on Nightly 46.0a1 or Firefox 40.0.

1. Please try testing with a clean profile https://support.mozilla.org/en-US/kb/refresh-firefox-reset-add-ons-and-settings 

2. Please try testing with your addons disabled https://support.mozilla.org/en-US/kb/disable-or-remove-add-ons
Flags: needinfo?(sworddragon2)
(Reporter)

Comment 10

3 years ago
(In reply to Justin - QA from comment #9)
> I can not reproduce this issue as my sort for username places testa before
> testb when selecting sort by username before adding test a on Nightly 46.0a1
> or Firefox 40.0.

Do you have sorted by Username before adding any of testa and testb and also added testb before testa? If not you maybe want to make a look again at the reproduction steps.
Flags: needinfo?(sworddragon2) → needinfo?(jwilliams)
Here is the screencast of my testing. Please inform me if I did any steps wrong.

http://www.screencast.com/t/PnmwZp8QBNst
Flags: needinfo?(jwilliams) → needinfo?(sworddragon2)
(Reporter)

Comment 12

3 years ago
Step 3 was missing but since you had no logins in this test you have possibly skipped it for this reason. But besides from this it looked correct and the bug did indeed not trigger in your case. I have now tested what happens if I reproduce this issue by ignoring step 3 too (even with my normal profile that contains several credentials): The bug did then not appear for me too. Maybe this bug was then triggered because searching for "https://bugzilla.mozilla.org" returned already my normal login.
Flags: needinfo?(sworddragon2)

Comment 13

3 years ago
Sworddragon2 - do we still believe we have a bug here? There might be something here, but I'm inclined to say that unless we can successfully reduce this to a minimal testcase or demonstrate some underlying pathology in our list sorting, that we won't be able to act on this.
Flags: needinfo?(sworddragon2)
(Reporter)

Comment 14

3 years ago
The bug seems to have a special condition but I can't tell what it is. Maybe somebody else is able to reproduce this issue or to provide more stable reproduction steps.
Flags: needinfo?(sworddragon2)
Component: Untriaged → Password Manager
Product: Firefox → Toolkit
set the proper component so that a dev can take a look at this.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Whiteboard: [passwords:management]
I agree this is a bug.
Status: NEW → RESOLVED
Last Resolved: 3 years ago2 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 211352
You need to log in before you can comment on or make changes to this bug.