User:Standard8/MailNews Password Manager Migration: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(Initial version mostly moved and updated from User:Standard8)
 
(→‎Current Status: Start adding protocol/url map of what provides what)
Line 8: Line 8:
* TB UI patch needs updating {{bug|239131}}
* TB UI patch needs updating {{bug|239131}}
* Backend-changes required (see below)
* Backend-changes required (see below)
== Backend changes/information ==
=== Protocols versus urls registered within MailNews ===
Ones I'm worried about:
{|border=1
! Protocol !! Handler !! Url !! Url derived from nsMsgMailNewsUrl?
|-
| ldap || [http://mxr.mozilla.org/seamonkey/source/directory/xpcom/base/src/nsLDAPProtocolHandler.cpp nsLDAPProtocolHandler.cpp] || nsLDAPUrl || No
|-
| ldaps || ? || nsLDAPUrl? || No
|-
| imap || [http://mxr.mozilla.org/seamonkey/source/mailnews/imap/src/nsImapService.cpp nsImapService.cpp] || ? || ?
|-
| news || [http://mxr.mozilla.org/seamonkey/source/mailnews/news/src/nsNntpService.cpp nsNntpService.cpp] || ? || ?
|-
| nntp || [http://mxr.mozilla.org/seamonkey/source/mailnews/news/src/nsNntpService.cpp nsNntpService.cpp] || ? || ?
|-
| snews || [http://mxr.mozilla.org/seamonkey/source/mailnews/news/src/nsNntpService.cpp nsNntpService.cpp] || ? || ?
|-
| mailto || [http://mxr.mozilla.org/seamonkey/source/mailnews/compose/src/nsSmtpService.cpp nsSmtpService.cpp] || ? || ?
|-
| smtp || ? || ? || ?
|-
| smtps || ? || ? || ?
|-
| pop || [http://mxr.mozilla.org/seamonkey/source/mailnews/local/src/nsPop3Service.cpp nsPop3Service.cpp] || ? || ?
|-
| pops || ? || ? || ?
|-
| mailbox || [http://mxr.mozilla.org/seamonkey/source/mailnews/local/src/nsMailboxService.cpp nsMailboxService.cpp] || ? || ?
|-
|}
Others:
* [http://mxr.mozilla.org/seamonkey/source/mailnews/addrbook/src/nsAddbookProtocolHandler.cpp addbook]
** Address book printing.
* [http://mxr.mozilla.org/seamonkey/source/mailnews/base/src/nsCidProtocolHandler.cpp cid]
** Something to do with [http://mxr.mozilla.org/seamonkey/source/mailnews/compose/src/nsMsgSend.cpp#2120 generating HTML]


== Current Problems ==
== Current Problems ==

Revision as of 14:51, 23 June 2008

Current Status

  • Main Core changes in place (PW migration, nsIAuthPrompt handling).
    • May require further changes to PW migration routines
  • Patches for SM Prefs, tools menu, front-end bug 390025
    • May be out-of-date
  • Build config & installer patch on bug 239131, need to get reviews when a bit closer.
  • TB UI patch needs updating bug 239131
  • Backend-changes required (see below)

Backend changes/information

Protocols versus urls registered within MailNews

Ones I'm worried about:

Protocol Handler Url Url derived from nsMsgMailNewsUrl?
ldap nsLDAPProtocolHandler.cpp nsLDAPUrl No
ldaps ? nsLDAPUrl? No
imap nsImapService.cpp ? ?
news nsNntpService.cpp ? ?
nntp nsNntpService.cpp ? ?
snews nsNntpService.cpp ? ?
mailto nsSmtpService.cpp ? ?
smtp ? ? ?
smtps ? ? ?
pop nsPop3Service.cpp ? ?
pops ? ? ?
mailbox nsMailboxService.cpp ? ?

Others:

Current Problems

  • News logins probably won't migrate properly.
  • Mail logins may not migrate properly if username has been changed since account was created (the uri uses the original account username) - or if they do, it'll be confusing to the user.
  • Need to check consistency of nsIURI::path for the mailnews uris.
  • Need to provide smtp protocol handler for mailnews (and any others that are missing).
  • Need secure version of LDAP protocol handler.
  • Some incorrect refcount or something causing cycle collector issues?

Notes to self

  • mailbox = pop3
  • ldap/ldaps is correct

Current formats of urls

Site Username nsIAuthPrompt Calls
imap://username@host username PromptPassword
ldap://localhost:389/dc=test <> PromptPassword
mailbox://username@host username PromptPassword
smtp://username@host <> PromptPassword and PromptUsernameAndPassword
news://localhost:119/#password
news://localhost:119/#username		 <>
<>		 Prompt and PromptPassword

Proposed new format

Hostname (site) httpRealm username nsIAuthPrompt Calls
imap://host imap://host username PromptPassword
ldap://localhost ldap://localhost/dc=test username +++ PromptPassword
mailbox://host mailbox://host username PromptPassword
smtp://host smtp://host username PromptPassword and PromptUsernameAndPassword
news://localhost/ news://localhost/ username PromptUsernameAndPassword

Misc Notes

  • formSubmitURL is always not set.
  • httpRealm is always set as you must set on of httpRealm or formSubmitURL (in LDAP case we need this anyway).
  • +++ username for LDAP won't be used straight away as it needs other bugs fixing first.
Retrieved from "https://wiki.allizom.org/index.php?title=User:Standard8/MailNews_Password_Manager_Migration&oldid=97300"