Skip to content

Scoped WebAuthn: web changes#36784

Merged
Joerger merged 4 commits intomasterfrom
joerger/scoped-webauthn-web-ui
Jan 22, 2024
Merged

Scoped WebAuthn: web changes#36784
Joerger merged 4 commits intomasterfrom
joerger/scoped-webauthn-web-ui

Conversation

@Joerger
Copy link
Copy Markdown
Contributor

@Joerger Joerger commented Jan 17, 2024

Pass MFA challenge scope and allow-reuse from the WebUI.

Part of the implementation for #35185

Based off #36665

@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 no-changelog Indicates that a PR does not require a changelog entry backport/branch/v16 and removed backport/branch/v16 labels Jan 17, 2024
@Joerger Joerger force-pushed the joerger/scoped-webauthn-web-ui branch 3 times, most recently from 5bcbeef to b4463a3 Compare January 17, 2024 20:47
@Joerger Joerger force-pushed the joerger/scoped-webauthn-web-ui branch from b4463a3 to ae25f2c Compare January 17, 2024 21:11
@Joerger
Copy link
Copy Markdown
Contributor Author

Joerger commented Jan 19, 2024

@ibeckermayer @ryanclark Friendly ping for review, we are trying to put this into v15.0.0

@Joerger Joerger force-pushed the joerger/scoped-webauthn-web-ui branch from 7d41dbb to 15ae37c Compare January 19, 2024 23:53
Comment thread web/packages/teleport/src/services/auth/auth.ts Outdated
Comment thread lib/web/mfa.go Outdated
@Joerger Joerger enabled auto-merge January 22, 2024 18:03
@Joerger Joerger added this pull request to the merge queue Jan 22, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Jan 22, 2024
@Joerger Joerger force-pushed the joerger/scoped-webauthn-web-ui branch from 2e229aa to 60ab991 Compare January 22, 2024 19:19
@Joerger Joerger enabled auto-merge January 22, 2024 19:44
@Joerger Joerger added this pull request to the merge queue Jan 22, 2024
Merged via the queue into master with commit 80ef114 Jan 22, 2024
@Joerger Joerger deleted the joerger/scoped-webauthn-web-ui branch January 22, 2024 20:22
@public-teleport-github-review-bot
Copy link
Copy Markdown

@Joerger See the table below for backport results.

Branch Result
branch/v15 Failed

Joerger added a commit that referenced this pull request Jan 23, 2024
* Add mfa challenge Scope and AllowReuse to web createAuthenticateChallengeHandler.

* Pass MFA challenge scope and allow-reuse from WebUI for mfa challenge requests.

* Address comments.
github-merge-queue Bot pushed a commit that referenced this pull request Jan 23, 2024
* Scoped WebAuthn: web changes (#36784)

* Add mfa challenge Scope and AllowReuse to web createAuthenticateChallengeHandler.

* Pass MFA challenge scope and allow-reuse from WebUI for mfa challenge requests.

* Address comments.

* Make challengeScope optional for ReAuthenticate to fix /e references.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants