Skip to content

[v15] Make Web Add User prompt for MFA just once#37202

Merged
Joerger merged 1 commit intobranch/v15from
joerger/v15/reuse-mfa-web-add-user
Jan 24, 2024
Merged

[v15] Make Web Add User prompt for MFA just once#37202
Joerger merged 1 commit intobranch/v15from
joerger/v15/reuse-mfa-web-add-user

Conversation

@Joerger
Copy link
Copy Markdown
Contributor

@Joerger Joerger commented Jan 24, 2024

Backport #37067 to branch/v15

* Allow webauthnResponse to be passed into API requests; Prompt for MFA before creating a user in the WebUI.

* Skip MFA check if webauthn is not configured.

* Add getWebauthnResponseForAdminAction helper function.

* Remove unused admin action name.

* Fix lint.

* Check for specific error.

* Address comments.

* Relax admin action mfa enforcement expectation, in line with #37136.
@github-actions
Copy link
Copy Markdown
Contributor

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@Joerger Joerger added the no-changelog Indicates that a PR does not require a changelog entry label Jan 24, 2024
@Joerger Joerger enabled auto-merge January 24, 2024 21:01
@Joerger Joerger added this pull request to the merge queue Jan 24, 2024
Merged via the queue into branch/v15 with commit 3aec256 Jan 24, 2024
@Joerger Joerger deleted the joerger/v15/reuse-mfa-web-add-user branch January 24, 2024 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/sm ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants