Created attachment 287399 [details] [diff] [review] The fix Toolkit's password manager makes (good) use of the ioService to get nsIURIs etc. For the LDAP parts, we need a nsLDAPProtocolHandler. We already have one in the experimental code, which is also used for the nsIChannel implementation. As we don't want the nsIChannel implementation at the moment, I'm proposing that we build with the protocol handler and just error out if anyone tries to create a new channel base on ldap. We also should implement GetHostPort for nsLDAPURL I'm not sure if we will need it or not, but I think it should be there. Attaching patch.
Comment on attachment 287399 [details] [diff] [review] The fix thx, Mark
Patch checked in -> fixed.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.