-
Notifications
You must be signed in to change notification settings - Fork 280
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
WebAuthn registration failure #57
Comments
Hey @rhawdor, |
Hey @paulwer !
So, to answer, no the error was not present on KC 21.0.1. |
i guess this is due to the following change in keycloak: @lukin should we support the old and the new way for backwards compatibility? |
I have this error too and this is annoying :/ |
a workaround for now would be to stay on 21 from my pov, if this is an option😕 |
it's not really an option :/ |
i dont know. tipp: look at the official integration mentioned beforw, what they changed...maybe this helps |
I can try myself, but i dont think it's gonna be easy and fast, i'm not really familiar with typescript and tailwind |
from my pov its an issue from mapping internal data by ftl and output it to the script. |
Where is the mapping behavior ? |
the line, which is failing according to the error message is: https://github.com/lukin/keywind/blob/master/theme/keywind/login/webauthn-register.ftl#L47C31-L47C50 the variable within keycloak before was: a string separeted by "," and afterwards its a List in java. Maybe try something like this:
(means: if its an array => join it to string, seperated with "," OR if its anything else (probably a string, from versions before) => just display it as string in the html) and here is the line of official keycloak: |
@paulwer Cheers man, this solved it for me. I've opened up a pull request with that line replaced |
Hello !
I recently updated to Keycloak 22 (more precisely 22.0.1, was using version 21 before), and I think I spotted a bug on WebAuthn security key registration.
Could this be related to #20831 or am I missing something ?
The text was updated successfully, but these errors were encountered: