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

NAS-133899 / 25.10 / Reduce number of 2FA public endpoints #15533

Merged
merged 2 commits into from
Jan 30, 2025

Conversation

anodos325
Copy link
Contributor

This commit removes several 2FA-related public endpoints from the API as they are unused by the UI team and are of dubious benefit.

The API response for resetting a user's 2FA token now includes the token in the response. This simplifies the task for API users to generate a 2FA token for a user and presenting it for use by removing the necessity to call auth.me after generating the token.

This commit removes several 2FA-related public endpoints from the
API as they are unused by the UI team and are of dubious benefit.

The API response for resetting a user's 2FA token now includes
the token in the response. This simplifies the task for API
users to generate a 2FA token for a user and presenting it for
use by removing the necessity to call `auth.me` after generating
the token.
@bugclerk
Copy link
Contributor

@bugclerk bugclerk changed the title Reduce number of 2FA public endpoints NAS-133899 / 25.10 / Reduce number of 2FA public endpoints Jan 30, 2025
@anodos325 anodos325 requested a review from yocalebo January 30, 2025 21:46
@anodos325 anodos325 merged commit ac1f80b into master Jan 30, 2025
2 checks passed
@anodos325 anodos325 deleted the improve-2fa branch January 30, 2025 22:29
@bugclerk
Copy link
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Jan 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants