matrix-spec-proposals
matrix-spec-proposals copied to clipboard
MSC1998: Two-Factor Authentication Providers
@cyphar I've flagged this as ready to review. If that's incorrect, please let me know.
I haven't ready the whole thing yet, but the detail is massively appreciated!
I hope I didn't go overboard, I ended up copying the structure of the actual spec docs but maybe I should've been a bit more terse?
@turt2live I've made the corrections you mentioned. One thing with using DELETE is that now it makes slightly less semantic sense to disable only some of the two-factor providers (then again, POST /disable is quite a bit uglier). If only there was a DELETE-PATCH. :wink:
@dainnilsson please move your comments to threads on the diff so people can reply.
@dainnilsson please move your comments to threads on the diff so people can reply.
Done!
Any updates on this? I would like to have this merged as I would love to use WebAuthn as second factor when authenticating.
Same question.
Any updates on this? I would like to have this merged as I would love to use WebAuthn as second factor when authenticating.
I haven't had much time to review this after I first posted it. It is still a feature I'd like (I still use Matrix daily and wish we'd have 2FA) but it seems that WebAuthn would require a more flexible design than the pretty basic one I came up with here -- this feature would almost certainly turn out better if it's handled by someone with more experience dealing with authentication.
Isn't this work effectively overshadowed by the OIDC initiative the matrix foundation is currently following?
From what I know, it'd essentially move authentication to somewhere else, which then has to implement webauthn instead. (The focus being that, by default, a server will use matrix-authentication-service as an internal OIDC provider, which already has an issue open about this: https://github.com/matrix-org/matrix-authentication-service/issues/18)
yes, OIDC would obviate the need for this. For now we'll keep it open until we are fully committed to OIDC.