msyavuz commented on PR #33255:
URL: https://github.com/apache/superset/pull/33255#issuecomment-2909513083
> using OAUTH and AUTH_USER_REGISTRATION = True login breaks, it seems
RECAPTCHA_PUBLIC_KEY is now required, although this is a good measure it was
not a required config before.
@dpgaspar I think it was required when registration is enabled but it was
implicit, Recaptcha would be broken and you couldn't register. I am not sure
what was the intended flow with fab as
https://flask-appbuilder.readthedocs.io/en/latest/user_registration.html
doesn't mention oauth.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org
