docs: Remove references to BrowserID #100
Conversation
Closes: SYNC-4209
source/glossary.rst
Outdated
| @@ -58,7 +58,7 @@ Glossary | |||
| This is the only secret shared with the client and is different for each | |||
| auth token. | |||
|
|
|||
| Generation Number | |||
| Generation Number (obsolete) | |||
There was a problem hiding this comment.
The docs in this repo seem to largely predate FxA's use of oauth so even though this references browserid here I'm not entirely sure generation has been completely obsoleted. We still have code in the oauth path that checks for generation as a fallback to keys_changed_at.
The FxA /v1/verify endpoint tokenserver uses claims to potentially return generation in its response but it's not a required field, so we have an Option<i64> for it.
So we should maybe punt on marking this as obsolete for now? I'm also a bit confused because such things like keys_changed_at are basically contents of an opaque token with the oauth impl., but apparently a generation value that our oauth verifier might be utilizing is not from the token itself.
There was a problem hiding this comment.
Yeah, totally fair. It wasn't super clear to me either, so I had doubts. I'll remove the (obsolete) tag.
Description
Remove references to active use of BrowserID except for legacy information.
Issue(s)
Closes: SYNC-4209