password manager fails to remove entries for which the url has a trailing slash

RESOLVED INCOMPLETE

Status

SeaMonkey
Passwords & Permissions
RESOLVED INCOMPLETE
14 years ago
7 years ago

People

(Reporter: Matthew Buckett, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3a) Gecko/20021212
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3a) Gecko/20021212

When attempting to remove a password associated with a host that has a trailing
slash (/) it fails. A password associated with the string
"ftp://somehost.domain/pub/" cannot be removed. This is because when the string
is tested a trailing slash is remove and so doesn't match. By adding an extra
trailing slash to the URL be removed.

The offending code is in: extensions/wallet/src/singsign.cpp

If this is considered a feature could it please be better documented in the
JavaScript interface please.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.




Here is some JavaScript that shows the bug.

    var passwordComponent = Components.classes[
	"@mozilla.org/passwordmanager;1"
    ].getService();
      
    var gPasswordManager = passwordComponent.QueryInterface(
	Components.interfaces.nsIPasswordManager
    );

    gPasswordManager.addUser("ftp://host/dir/", "user", "password");
    gPasswordManager.removeUser("ftp://host/dir/", "user");
Product: Browser → Seamonkey
Assignee: dveditz → nobody

Comment 1

9 years ago
Still reproduceable with Seamonkey 1.1.8.
Let's hope bug 390025 will fix this.
QA Contact: tpreston

Updated

9 years ago
Depends on: 390025
QA Contact: privacy

Updated

7 years ago
Whiteboard: [obsoleted when bug 588421 lands]

Comment 2

7 years ago
Now that we are using the toolkit login manager with totally new code I doubt this problem still exists. Closing as INCOMPLETE. Please reopen if you can reproduce this with SeaMonkey 2.0.10 or 2.1b2pre.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → INCOMPLETE
Whiteboard: [obsoleted when bug 588421 lands]
You need to log in before you can comment on or make changes to this bug.