Closed Bug 461341 Opened 15 years ago Closed 15 years ago

MailNews usernames need decoding on signons file upgrade

Categories

(Toolkit :: Password Manager, defect)

Product:
Toolkit
Component:
Password Manager
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9.1b2

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

The fix
5.54 KB, patch
Dolske
: review+
Details | Diff | Splinter Review
Attached patch The fix — — Splinter Review 
In Firefox, I get usernames (from my webmail) which are "foo+bar".

In my test Thunderbird using the new profile manager, the equivalent mailnews logins are shown as "foo%2Bbar".

This is wrong and will be confusing to the user, we should be decoding the mailnews usernames when we migrate.

Attaching patch that will fix this, including updated unit tests.
Attachment #344466 - Flags: review?(dolske)
Attachment #344466 - Flags: review?(dolske) → review+
Checked in: http://hg.mozilla.org/mozilla-central/rev/d0251e736541
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.