Closed Bug 960324 Opened 6 years ago Closed 6 years ago

Use "BrowserID" in tokenserver authz header, not "Browser-ID"

Categories

(Firefox for Android :: Android Sync, defect)

defect
Not set

Tracking

()

VERIFIED FIXED

People

(Reporter: rfkelly, Assigned: rfkelly)

References

(Blocks 1 open bug)

Details

(Whiteboard: [qa-])

Attachments

(1 file)

Per Bug 957503
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 #8361507 - Flags: review?(nalexander)
Comment on attachment 8361507 [details] [diff] [review]
android-browserid-auth-header.diff

Review of attachment 8361507 [details] [diff] [review]:
-----------------------------------------------------------------

I'll land this in s-c.
Attachment #8361507 - Flags: review?(nalexander) → review+
https://hg.mozilla.org/services/services-central/rev/33c0a093085f
Assignee: nobody → rfkelly
Status: NEW → ASSIGNED
Whiteboard: [fixed in services][qa-]
https://hg.mozilla.org/mozilla-central/rev/33c0a093085f
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in services][qa-] → [qa-]
Quick checked the code.
And since it has hit m-c...
Status: RESOLVED → VERIFIED
Product: Android Background Services → Firefox for Android
You need to log in before you can comment on or make changes to this bug.