-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Minor cleanups from PR 1270 review #1291
Comments
this is a very minor nice-to-have issue, can be addressed in a milestone later than wd-03 or not at all. |
@emlun To look to see if this is still in play |
Okay, here goes... 🙂
I'm not sure it's worth introducing a formal term for this, I think it's clear enough without it.
Same here, I think "first time" is clear enough without needing to introduce a formal term.
This one I think might still be relevant. "Multi-factor" would technically be more accurate, but on the other hand "first-factor" highlights that it can be used as the first step of an authentication procedure. I'm not sure if there's one that's clearly "better" than the other.
This seems unnecessary to me as it doesn't really say anything new. Maybe if we were using "bootstrap" elsewhere in the spec, to tie them together, but we currently don't. @equalsJeffH thoughs on that? |
thx for your thoughts @emlun, ISTM this is a nice-to-have and puntable to a later spec version, or not at all. |
This was recently obsoleted by #2140 and #2138. I still think the other points are unnecessary as described above. I think we can close this issue. |
Unresolved discussions from #1270 (review) :
Improve authenticator taxonomy section #1270 (comment)
Improve authenticator taxonomy section #1270 (comment)
Improve authenticator taxonomy section #1270 (comment)
Improve authenticator taxonomy section #1270 (comment)
The text was updated successfully, but these errors were encountered: