Closed
Bug 959476
Opened 11 years ago
Closed 11 years ago
Remove X-Conditions-Accepted from tokenserver 1.0 spec
Categories
(Cloud Services Graveyard :: Server: Token, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: rfkelly, Assigned: rfkelly)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file)
2.14 KB,
patch
|
telliott
:
review+
|
Details | Diff | Splinter Review |
We no longer need to enforce ToS acceptance at the server level, and have no plans to ship anything that needs this functionality from the tokenserver. Refactoring the backend to support this in conjunction with generation/fingerprint stuff is looking pretty yuck.
Can we just yank this from the 1.0 spec and revisit if we need it down the road?
Attachment #8359603 -
Flags: review?(telliott)
Assignee | ||
Comment 1•11 years ago
|
||
(IIRC it was a pretty last-minute addition to begin with, for the rush-rush-rush AitC deadline way back when)
Comment 2•11 years ago
|
||
Comment on attachment 8359603 [details] [diff] [review]
tokenserver-no-conditions.diff
Review of attachment 8359603 [details] [diff] [review]:
-----------------------------------------------------------------
Yeah, if it's getting in the way, keeping it around is low priority for now.
Attachment #8359603 -
Flags: review?(telliott) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Docs change in https://github.com/mozilla-services/docs/commit/5319b1daca67e08c7f73b8983c0d433a41cff0a9
Assignee: nobody → rfkelly
Updated•11 years ago
|
Whiteboard: [qa-]
Assignee | ||
Comment 4•11 years ago
|
||
it's done
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 5•11 years ago
|
||
Yep verified here: https://github.com/mozilla-services/docs/tree/master/source/token
Status: RESOLVED → VERIFIED
Updated•2 years ago
|
Product: Cloud Services → Cloud Services Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•