Closed Bug 776474 Opened 12 years ago Closed 12 years ago

Make twitter account setup less fragile

Categories

(Thunderbird :: Instant Messaging, defect)

15 Branch
x86
All
defect
Not set
normal

Tracking

(thunderbird15 fixed, thunderbird16 fixed)

RESOLVED FIXED
Thunderbird 17.0
Tracking Status
thunderbird15 --- fixed
thunderbird16 --- fixed

People

(Reporter: thomas, Assigned: florian)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:14.0) Gecko/20100101 Firefox/14.0.1
Build ID: 20120713224758

Steps to reproduce:

I tried to add more than one Twitter Account, but login was not possible for the second account.


Actual results:

I can only use one Twitter Account in the new Chat Tab. After authentication via Twitter API I get the following error:

"Error: Username mismatch" 


Expected results:

It would be nice, if I could use multiple Twitter Accounts
Were you able to create those 2 accounts in the account manager? If yes, then one of the accounts did not work?
Component: Account Manager → Instant Messaging
Ok, know it works for me too.. the problem was, that TB is case-sensitive and throws the error "User mismatch", if the username is not correct. Twitter is not as restrict here (not case-sensitive!), so maybe it would be better to change this in TB as well.
Attached patch PatchSplinter Review
Changes included in this patch:
- fix the case in account names instead of refusing to connect the account (fixes what's described in comment 2)
- ignore existing cookies when opening the authorization dialog. I suspect this fixes what was initially described in comment 0.
- prefill the input box in the authorization dialog.
Assignee: nobody → florian
Attachment #644958 - Flags: review?(clokep)
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Summary: Multiple Twitter Accounts not working → Make twitter account setup less fragile
Comment on attachment 644958 [details] [diff] [review]
Patch

This looks good! :) Thanks for fixing these annoyances.
Attachment #644958 - Flags: review?(clokep) → review+
Attachment #644958 - Flags: approval-comm-beta?
Attachment #644958 - Flags: approval-comm-aurora?
Attachment #644958 - Flags: approval-comm-beta?
Attachment #644958 - Flags: approval-comm-beta+
Attachment #644958 - Flags: approval-comm-aurora?
Attachment #644958 - Flags: approval-comm-aurora+
comm-central: https://hg.mozilla.org/comm-central/rev/62a941631475
comm-aurora: https://hg.mozilla.org/releases/comm-aurora/rev/3c7e60ef4858
comm-beta: https://hg.mozilla.org/releases/comm-beta/rev/4c63a7a39781
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
OS: Linux → All
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
Backed out of comm-beta since we landed on a SeaMonkey relbranch (oops).

Re-landed on comm-beta as: https://hg.mozilla.org/releases/comm-beta/rev/7ebadaff038e
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: