The default bug view has changed. See this FAQ.

Add support for the PLAINTEXT signature method to oauth.jsm

RESOLVED FIXED in Thunderbird 18.0

Status

Thunderbird
FileLink
--
enhancement
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: florian, Assigned: Tom Thompson)

Tracking

(Blocks: 1 bug)

Trunk
Thunderbird 18.0
Dependency tree / graph

Thunderbird Tracking Flags

(thunderbird16 fixed, thunderbird17 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
The PLAINTEXT OAuth signature method is required to add Spideroak support for Filelink, and is also needed to fix bug 764312 (sending files with a name containing the '(' or ')' characters through Ubuntu One).

A patch for this has been developed and reviewed in bug 744035 for Spideroak, but it would be useful for Ubuntu One to land this before the Spideroak support is finished, so I'm splitting this out to a separate bug per Mark's request.
(Reporter)

Comment 1

5 years ago
Created attachment 664103 [details] [diff] [review]
Patch (from bug 744035)

This patch comes from bug 744035. It's an almost exact copy of attachment 636163 [details] [diff] [review].
I only fixed a coding style nit I had (renamed the "verifier" parameter to "aVerifier") and added a commit message (that lists Tom Thompson as the author).

David Bienvenu already reviewed that patch in bug 744035. I did a drive by review there of a previous version of the patch (I'm the original author of that OAuth code), and my review comments have been addressed.

I think we should take this for Thunderbird 17 to fix bug 764312, and possibly for Thunderbird 16 too.

After this lands, someone should verify that bug 764312 is actually fixed, and that the Dropbox add-on still works.
Attachment #664103 - Flags: review+
Attachment #664103 - Flags: approval-comm-beta?
Attachment #664103 - Flags: approval-comm-aurora?
(Reporter)

Comment 2

5 years ago
https://hg.mozilla.org/comm-central/rev/6eea3c05df45
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
Comment on attachment 664103 [details] [diff] [review]
Patch (from bug 744035)

Mike says this works fine with Dropbox, so a=me.
Attachment #664103 - Flags: approval-comm-beta?
Attachment #664103 - Flags: approval-comm-beta+
Attachment #664103 - Flags: approval-comm-aurora?
Attachment #664103 - Flags: approval-comm-aurora+
Checked in:

https://hg.mozilla.org/releases/comm-aurora/rev/05cd7c19db4e
https://hg.mozilla.org/releases/comm-beta/rev/5f3a3fa7a27f
status-thunderbird16: --- → fixed
status-thunderbird17: --- → fixed

Updated

2 years ago
Depends on: 1109178
You need to log in before you can comment on or make changes to this bug.