p2-inc / keycloak-magic-link

Magic Link Authentication for Keycloak
https://phasetwo.io
Other
222 stars 44 forks source link

EmailOtpAuthenticator does not work with "Alternative" #31

Closed srt closed 1 year ago

srt commented 1 year ago

If set to "Alternative" instead of "Required" EmailOtpAuthenticator does not succeed and returns context.attempted() instead of context.success().

Example flow to reproduce the issue:

image

xgp commented 1 year ago

Seems like a bug. Let me run a couple of additional use cases to figure out why I added that conditional.

xgp commented 1 year ago

fixed in 50f2c0e

srt commented 1 year ago

Thanks for the quick fix!