793749 - Add support for the PLAINTEXT signature method to oauth.jsm

Status: RESOLVED FIXED

(Thunderbird :: FileLink, enhancement)

Description

[Florian Quèze [:florian]]

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.

Comment 1

[Florian Quèze [:florian]]

Attachment: 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.

Comment 2

[Florian Quèze [:florian]]

https://hg.mozilla.org/comm-central/rev/6eea3c05df45

Comment 3

[Mark Banner (:standard8)]

Comment on attachment 664103 [details] [diff] [review]
Patch (from bug 744035)

Mike says this works fine with Dropbox, so a=me.

Comment 4

[Mark Banner (:standard8)]

Checked in:

https://hg.mozilla.org/releases/comm-aurora/rev/05cd7c19db4e
https://hg.mozilla.org/releases/comm-beta/rev/5f3a3fa7a27f