Skip to content

[v15] fix: Assert credentials individually on U2F devices#48403

Merged
codingllama merged 1 commit intobranch/v15from
codingllama/backport/45289-v15
Nov 5, 2024
Merged

[v15] fix: Assert credentials individually on U2F devices#48403
codingllama merged 1 commit intobranch/v15from
codingllama/backport/45289-v15

Conversation

@codingllama
Copy link
Copy Markdown
Contributor

@codingllama codingllama commented Nov 4, 2024

Backport #45289 to branch/v15.

Changelog: Fixes a rare "internal error" on older U2F authenticators when using tsh

#44912

* Simulate "internal error" on multiple credentials

* fix: Assert credentials individually on U2F devices

* Use bytes.Repeat

* Comment on U2F and libfido2.ErrUserPresenceRequired

* Move errorOnUnknownCredential failure after the "tap"
@aws-amplify-us-west-2
Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-48403.d1v2yqnl3ruxch.amplifyapp.com

@codingllama
Copy link
Copy Markdown
Contributor Author

Friendly ping @smallinsky ?

@codingllama codingllama enabled auto-merge November 5, 2024 13:26
@codingllama codingllama added this pull request to the merge queue Nov 5, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 5, 2024
@codingllama codingllama added this pull request to the merge queue Nov 5, 2024
Merged via the queue into branch/v15 with commit 72b2783 Nov 5, 2024
@codingllama codingllama deleted the codingllama/backport/45289-v15 branch November 5, 2024 14:45
@camscale camscale mentioned this pull request Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants