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

Make WebAuthnAuthentication Serializable #16273

Open
jzheaux opened this issue Dec 12, 2024 · 1 comment · May be fixed by #16285
Open

Make WebAuthnAuthentication Serializable #16273

jzheaux opened this issue Dec 12, 2024 · 1 comment · May be fixed by #16285
Labels
in: web An issue in web modules (web, webmvc) type: bug A general bug
Milestone

Comments

@jzheaux
Copy link
Contributor

jzheaux commented Dec 12, 2024

WebAuthnAuthentication implements Serializable, but cannot be serialized due to PublicKeyCredentialUserEntity, one of its members, not being serializable.

@jzheaux jzheaux added in: web An issue in web modules (web, webmvc) type: bug A general bug labels Dec 12, 2024
@jzheaux jzheaux added this to the 6.4.x milestone Dec 12, 2024
@justincranford
Copy link

I reported a similar issue, but more broadly it affects registration too.
#16328

WebAuthn + Redis doesn't work out-of-the-box...

  1. WebAuthn classes lack Serializable interface, and Redis defaults to JdkSerializationRedisSerializer.
  2. WebAuthn has mixins for GenericJackson2JsonRedisSerializer, but I had to add 12 extras because bundled mixins are partially incomplete or missing.

Workarounds to create an ObjectMapper for GenericJackson2JsonRedisSerializer which can serialize the WebAuthn registration and authentication request objects.
#16328 (comment)

Workarounds to configure Redis with GenericJackson2JsonRedisSerializer.
#16328 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: web An issue in web modules (web, webmvc) type: bug A general bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants