Closed
Bug 1388804
Opened 8 years ago
Closed 6 years ago
Use Mailnew's OAuth2 module in Calendar Gdata
Categories
(Calendar :: Provider: GData, enhancement)
Calendar
Provider: GData
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: jorgk-bmo, Unassigned)
Details
I noticed that we have two almost identical OAuth2.jsm modules:
mailnews/base/util/OAuth2.jsm
calendar/providers/gdata/modules/OAuth2.jsm
Looks like both files were created in 2014 and then the mailnews version hasn't seen changes since 2015.
Maybe it's time to unify them.
Comment 1•8 years ago
|
||
The gdata provider used to support very old Gecko based applications like Postbox. Maybe this was the reason to include its own OAuth2 implementation. If this was the only reason than it might be time for cleanup because the required Gecko version is now 45 according to install.rdf.
Comment 2•6 years ago
|
||
The future would likely be to use the browser.identity API. I'm going to close this bug since I don't anticipate that we'll be changing this any time soon. Lightning is welcome to steal changes from the Provider regarding OAuth if necessary.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•