lukin / keywind

Keywind is a component-based Keycloak Login Theme built with Tailwind CSS
Apache License 2.0
737 stars 261 forks source link

Fix WebAuthN Issue #85

Open scheibling opened 4 months ago

scheibling commented 4 months ago

Fixes bug with WebAuthN registration that occured around v21, resolves #57, #76 . Backwards-compatible with earlier versions

Suggestion by @paulwer https://github.com/lukin/keywind/issues/57#issuecomment-1751943584

I've tested it with pre-21 and post-21 versions, seems to work well up to the newest version.

youss6f commented 3 months ago

Uncaught Reference Error: registerSecurityKey is not defined

scheibling commented 3 months ago

Uncaught Reference Error: registerSecurityKey is not defined

@youss6f That doesn't really look like something this PR would cause, are there any other error messages before that one that would indicate otherwise? Under which circumstances/with which versions are you getting this error?

cJlD2ENp4PoPQ commented 3 months ago

I had the same problem on Keycloak version 23.0.6

But I forgot to copy the " at the end of the line. After applying the whole change the registration worked as expected.

@youss6f Did you make the same misstake?