The default bug view has changed. See this FAQ.

Port Twitter API v1.1 from Instantbird to c-c

RESOLVED FIXED in Thunderbird 23.0


Instant Messaging
4 years ago
4 years ago


(Reporter: clokep, Assigned: clokep)


Thunderbird 23.0

Thunderbird Tracking Flags

(thunderbird21+ fixed, thunderbird22 fixed, thunderbird-esr1721+ fixed)




(1 attachment)



4 years ago
See also bug 842183 and friends.

API v1 of twitter is being retired on May 7th [1]. Instantbird has been updated to v1.1 [2] (the newest version), these changes should be ported to Thunderbird (and probably into the newest version as possible, beta or aurora?) After this date, users of Thunderbird will be unable to connect to Twitter.

This also requires a change to http.jsm. (mconley has already tested this change and approved it.)

What is the status of Thunderbird releases? How can we get this out quickly? Should this be included in an ESR branch?

tracking-thunderbird21: --- → +
tracking-thunderbird-esr17: --- → 21+

Comment 1

4 years ago
Created attachment 732631 [details] [diff] [review]

This is a port of the Instantbird changeset [1], with the http.jsm changes also applied to mail/base/modules/http.jsm. Unfortunately I didn't have a c-c build that was anywhere close to up to date and this is going to take a while to this is currently "untested" (as part of TB, IB nightlies have been running with a similar patch for a while), but will be tested as soon as I can get my build running.

Assignee: nobody → clokep
Attachment #732631 - Flags: review?(florian)


4 years ago
Attachment #732631 - Flags: review?(mconley)
Comment on attachment 732631 [details] [diff] [review]

rs=me for the port from the Instantbird repository. The code has already been r+'ed by the relevant chat/ peers (including me) in

Would still be good to have Mike's review for mail/base/modules/http.jsm (he only gave feedback+ in the Instantbird bug).
Attachment #732631 - Flags: review?(florian) → review+
Comment on attachment 732631 [details] [diff] [review]

Review of attachment 732631 [details] [diff] [review]:

Yeah, this is totally fine. Let's roll with this.
Attachment #732631 - Flags: review?(mconley) → review+
Last Resolved: 4 years ago
Resolution: --- → FIXED
Comment on attachment 732631 [details] [diff] [review]

[Triage Comment]
Ok, lets get this on Aurora and Beta for some accelerated testing as we'll need this soon.
Attachment #732631 - Flags: approval-comm-beta+
Attachment #732631 - Flags: approval-comm-aurora+
status-thunderbird21: --- → fixed
status-thunderbird22: --- → fixed
Comment on attachment 732631 [details] [diff] [review]

[Triage Comment]
I've not heard of any issues, so lets push forward to ESR as well so that we're ready.
Attachment #732631 - Flags: approval-comm-esr17+
Comment on attachment 732631 [details] [diff] [review]

Unfortunately this patch doesn't apply cleanly to comm-esr17, can someone take a look at doing a patch before early next week? Thanks.
Attachment #732631 - Flags: approval-comm-esr17+ → approval-comm-esr17-
Ping, has anyone had time to do this?
Ok, I think I've unbitrotted this correctly and pushed it here:

If poss, please review/try it out, I'll be trying it in a bit once the builds complete.
I've tested as best as I can, and I still seem to receive tweets and things, so hopefully the merge was successful. If others can please test esr branch before we release on Tuesday, that'd be really useful.
status-thunderbird-esr17: --- → fixed
Target Milestone: --- → Thunderbird 23.0

Comment 12

4 years ago
(In reply to Mark Banner (:standard8) from comment #10)
> Ok, I think I've unbitrotted this correctly and pushed it here:
> If poss, please review/try it out, I'll be trying it in a bit once the
> builds complete.

This looks fine! Thanks for doing this. Sorry I couldn't get to this earlier, I was traveling without a computer for the past two weeks.
You need to log in before you can comment on or make changes to this bug.