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

[stable30] fix(webauthn): Increase database column for public key id #47249

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Aug 14, 2024

Backport of PR #47240

@backportbot backportbot bot requested review from kesselb, st3iny and susnux August 14, 2024 23:57
@backportbot backportbot bot added this to the Nextcloud 30.0.1 milestone Aug 14, 2024
@susnux susnux requested a review from skjnldsv August 15, 2024 00:06
@susnux susnux modified the milestones: Nextcloud 30.0.1, Nextcloud 30 Aug 15, 2024
@skjnldsv skjnldsv mentioned this pull request Aug 22, 2024
44 tasks
@susnux susnux force-pushed the backport/47240/stable30 branch from 17ca457 to d101e07 Compare August 22, 2024 09:54
@susnux susnux force-pushed the backport/47240/stable30 branch from d101e07 to 41c222b Compare August 22, 2024 11:34
@susnux susnux requested a review from nickvergessen August 22, 2024 11:35
* Resolves #34476

There is no maximum length defined in the standard,
most common the length is between 128 and 200 characters,
but as we store it not in plain data but base64 encoded the length can grow about 1/3.

We had a regression with 'Nitrokey 3' which created IDs with 196 byte length -> 262 bytes encoded base64.
So to be save we increase the size to 512 bytes.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@nickvergessen nickvergessen force-pushed the backport/47240/stable30 branch from 41c222b to aa383a8 Compare August 22, 2024 12:43
@skjnldsv skjnldsv merged commit 11d2240 into stable30 Aug 22, 2024
168 of 169 checks passed
@skjnldsv skjnldsv deleted the backport/47240/stable30 branch August 22, 2024 13:43
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