Skip to content
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

Rethink capability name /mfa-capable #133

Closed
michielbdejong opened this issue Sep 13, 2024 · 1 comment · Fixed by #136
Closed

Rethink capability name /mfa-capable #133

michielbdejong opened this issue Sep 13, 2024 · 1 comment · Fixed by #136

Comments

@michielbdejong
Copy link
Contributor

We can change the string /mfa-capable to something without a leading slash.
Also, the 'capable' is already implied by the fact that this string appears in a list of capabilities
And maybe we want to add what the server is capable of doing. It's not only capable of MFA, we want it to be able to securely and reliably enforce an MFA requirement on a share.
So maybe something like 'mfa-requirement' or 'enforce-mfa-requirements'?

@glpatcern
Copy link
Member

I think the bigger picture is well summarized in #121

@glpatcern glpatcern linked a pull request Sep 30, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants