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

Status: RESOLVED INCOMPLETE

(SeaMonkey :: Passwords & Permissions, defect)

Description

[Matthew Buckett]

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");

Comment 1

[Thomas Bertels]

Still reproduceable with Seamonkey 1.1.8.
Let's hope bug 390025 will fix this.

Comment 2

[Philip Chee]

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.