419590 - Review MailNews implementations of nsIProtocolHander and nsIURI

Status: RESOLVED WONTFIX

(MailNews Core :: Backend, defect, P2)

Description

[Mark Banner (:standard8)]

I've come up against various problems implementing the transfer to password manager for mailnews:

* Not all protocols have handlers, e.g. smtp, smtps, ldaps
* Some protocols have other handlers implemented, e.g. smtp is actually treated as mailto sometimes.
* nsIURI implementations are inconsistent e.g. nsLDAPURL doesn't return a "/" at the start of the path name and it needs to, and also doesn't actually return the full path.

This one may not be quite relevant to this bug but I'll mention it here anyway:

* Some of the uris stored in password manager actually are based on the rdf uris of the backend, and don't reflect the actual details of the current server (e.g. references original hostname, not the current server hostname).

I think this is probably going to be more of a tracking bug at the moment. I need to take these in turn and check our implementations of them.

Comment 1

[Mark Banner (:standard8)]

Due to the way we're implementing the switch-over to toolkit password manager, I no longer need this bug, so marking wontfix for the time being.

Should we later wish to fix or reimplement how areas of mailnews works wrt uris/passwords etc, then we can reopen this, or file new bugs.