Last Comment Bug 359882 - autoconfig e-mail address detection doesn't work with multiple identities
: autoconfig e-mail address detection doesn't work with multiple identities
: fixed1.8.1.1
Product: Core
Classification: Components
Component: Preferences: Backend (show other bugs)
: Trunk
: x86 Windows XP
-- normal (vote)
: ---
Assigned To: David :Bienvenu
: Benjamin Smedberg [:bsmedberg]
Depends on:
  Show dependency treegraph
Reported: 2006-11-07 15:14 PST by David :Bienvenu
Modified: 2007-03-13 03:18 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

proposed fix (1.63 KB, patch)
2006-11-07 15:15 PST, David :Bienvenu
mscott: superreview+
Details | Diff | Splinter Review
fix as checked in (1.71 KB, patch)
2006-11-28 09:42 PST, David :Bienvenu
mozilla: superreview+
Details | Diff | Splinter Review
patch for trunk/2.0 branch (1.36 KB, patch)
2006-11-28 09:57 PST, David :Bienvenu
no flags Details | Diff | Splinter Review

Description User image David :Bienvenu 2006-11-07 15:14:29 PST
If the default account has multiple identities, we can't find the e-mail address from the identity, because we don't parse the multiple identities. Patch upcoming
Comment 1 User image David :Bienvenu 2006-11-07 15:15:18 PST
Created attachment 244943 [details] [diff] [review]
proposed fix
Comment 2 User image David :Bienvenu 2006-11-28 09:42:21 PST
Created attachment 246807 [details] [diff] [review]
fix as checked in

there was a bug in the previous patch - we need to adjust the length of the id string as well - this is what I'll land. Carrying forward sr.
Comment 3 User image David :Bienvenu 2006-11-28 09:57:44 PST
Created attachment 246812 [details] [diff] [review]
patch for trunk/2.0 branch

this is a patch for the trunk/2.0 branch - prev patch was for 1.0 branch
Comment 4 User image David :Bienvenu 2006-11-28 10:05:02 PST
fixed on trunk and branches.
Comment 5 User image David :Bienvenu 2006-12-05 19:36:52 PST
for QA/testing, this change only affects autoconfig, and testing autoconfig is a bit of a challenge. But see this for some info:

Note You need to log in before you can comment on or make changes to this bug.