Closed
Bug 960322
Opened 12 years ago
Closed 12 years ago
Use "BrowserID" in tokenserver authz header, not "Browser-ID"
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
VERIFIED
FIXED
mozilla29
People
(Reporter: rfkelly, Assigned: rfkelly)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file)
|
537 bytes,
patch
|
rnewman
:
review+
|
Details | Diff | Splinter Review |
Per Bug 957503
Updated•12 years ago
|
Whiteboard: [qa?]
| Assignee | ||
Comment 1•12 years ago
|
||
The current dev tokenserver will accept the header in either format, but new deployments will only accept "BrowserID". Let's fix this up while it's still trivial to do so.
Attachment #8361506 -
Flags: review?(rnewman)
Comment 2•12 years ago
|
||
Comment on attachment 8361506 [details] [diff] [review]
desktop-browserid-auth-header.diff
Review of attachment 8361506 [details] [diff] [review]:
-----------------------------------------------------------------
I'll land this in s-c.
Attachment #8361506 -
Flags: review?(rnewman) → review+
Comment 3•12 years ago
|
||
Assignee: nobody → rfkelly
Status: NEW → ASSIGNED
Whiteboard: [qa?] → [fixed in services][qa-]
Comment 4•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in services][qa-] → [qa-]
Target Milestone: --- → mozilla29
Comment 5•12 years ago
|
||
Quick checked the code.
And since it has hit m-c...
Status: RESOLVED → VERIFIED
Updated•7 years ago
|
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•