Closed Bug 844175 Opened 7 years ago Closed 7 years ago

Twitter stays on Connecting: Requesting user timelines...

Categories

(Thunderbird :: Instant Messaging, defect, major)

21 Branch
defect
Not set
major

Tracking

(thunderbird21+ fixed)

RESOLVED FIXED
Thunderbird 22.0
Tracking Status
thunderbird21 + fixed

People

(Reporter: Paenglab, Assigned: florian)

References

Details

(Keywords: regression)

Attachments

(2 files)

Twitter is always in Connecting: Requesting user timelines... state. In console I have this:

Error: Component returned failure code: 0x8000ffff (NS_ERROR_UNEXPECTED) [nsIPrefBranch.getCharPref]
Source file: resource://gre/components/imIncomingServer.js
Line: 179

After disconnecting and connecting again I'm seeing this:

Fri Feb 22 2013 18:45:40
Error: NS_ERROR_ILLEGAL_VALUE: Illegal value
Source file: resource://app/modules/http.jsm
Line: 13

Last good was TB 21 from 16.2.2013. Because of build bustage the next build was from 19.2.2013 and this shows the issue. Maybe it has something to do with Bug 842183 which land in this time.

I tried this on Win7 and Linux.
The second error looks like it's from a fork in http.jsm between chat/ [1] and mail/ [2]:
Mail:
> function doXHRequest(aUrl, aHeaders, aPOSTData, aOnLoad, aOnError, aThis, aMethod)

Chat:
> function doXHRequest(aUrl, aHeaders, aPOSTData, aOnLoad, aOnError, aThis, aLogger)

We now pass aLogger, this gets considered (in Thunderbird) as aMethod and causes that error.

[1] http://mxr.mozilla.org/comm-central/source/chat/modules/http.jsm
[2] http://mxr.mozilla.org/comm-central/source/mail/base/modules/http.jsm
Same here - for old accounts that was worked before v21 uodate i got -

requesting user timelines..

but before there was a second dialog asking (usually annoying because i was activating it for mail already) for master password; for the same twi acc. i've created a new kinda account in bird - and i got:

Connecting: initiating authentication process 

but irc is running already in the same time not sure about fb.
Assignee: nobody → florian
Status: NEW → ASSIGNED
Attachment #719723 - Flags: review?(clokep)
Blocks: 842183
Comment on attachment 719723 [details] [diff] [review]
Sync the two http.jsm files

These changes look good and should get Twitter back on track.

I don't think I can actually review things in mail/base, however.
Attachment #719723 - Flags: review?(mconley)
Attachment #719723 - Flags: review?(clokep)
Attachment #719723 - Flags: review+
Severity: normal → major
Target Milestone: --- → Thunderbird 22.0
Version: unspecified → 21
Comment on attachment 719723 [details] [diff] [review]
Sync the two http.jsm files

Review of attachment 719723 [details] [diff] [review]:
-----------------------------------------------------------------

I'm alright with this. It blows that we have duplicated http.jsm, though. If both InstantBird and Thunderbird are using it, it belongs in toolkit.
Attachment #719723 - Flags: review?(mconley) → review+
https://hg.mozilla.org/comm-central/rev/361ace8a7590
http://hg.instantbird.org/instantbird/rev/237424fae13f
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Attached patch Bustage fixSplinter Review
(In reply to Florian Quèze [:florian] [:flo] from comment #7)
> Created attachment 721706 [details] [diff] [review]
> Bustage fix

https://hg.mozilla.org/comm-central/rev/94f3a8a2bdf7 (rs=mconley over IRC)
(In reply to Florian Quèze [:florian] [:flo] from comment #8)
> (In reply to Florian Quèze [:florian] [:flo] from comment #7)
> > Created attachment 721706 [details] [diff] [review]
> > Bustage fix
> 
> https://hg.mozilla.org/comm-central/rev/94f3a8a2bdf7 (rs=mconley over IRC)

And http://hg.instantbird.org/instantbird/rev/2b9937637ece
Comment on attachment 719723 [details] [diff] [review]
Sync the two http.jsm files

[Approval Request Comment]
Regression caused by (bug #): bug 842183
User impact if declined: Twitter completely unusable.
Testing completed (on c-c, etc.): Already landed for c-c and Instantbird.
Risk to taking this patch (and alternatives if risky): Taking only the change to twitter.js may be possible, but I'm not sure that would really reduce the risk.

Note: I'm requesting approval on only one attachment, but we need to port both 361ace8a7590 and 94f3a8a2bdf7 to comm-aurora.
Attachment #719723 - Flags: approval-comm-aurora?
Attachment #719723 - Flags: approval-comm-aurora? → approval-comm-aurora+
(In reply to Florian Quèze [:florian] [:flo] from comment #10)

> Note: I'm requesting approval on only one attachment, but we need to port
> both 361ace8a7590 and 94f3a8a2bdf7 to comm-aurora.

Pushed both at once as https://hg.mozilla.org/releases/comm-aurora/rev/0fd17de09e0e
You need to log in before you can comment on or make changes to this bug.