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

[Next][Bug] authRedirectBackTo session storage key not set #2529

Closed
cristiandean opened this issue Sep 5, 2019 · 4 comments

Comments

@cristiandean
Copy link
Contributor

commented Sep 5, 2019

After update the last release (v5.0.0-beta.8) we had a problem.

After the user authenticate in the Talk, the authentication happen with success, but the user receives the following message in the pop up: "authRedirectBackTo session storage key not set"

This is when we have a single provider enabled (OIDC).

@wyattjoh

This comment has been minimized.

Copy link
Member

commented Sep 10, 2019

To clarify @cristiandean, this is only in the Admin? I was unable to reproduce the issue on our side.

@cristiandean

This comment has been minimized.

Copy link
Contributor Author

commented Sep 10, 2019

@wyattjoh this is in embed stream. For reproduce you have to:

Enable OpenID provider with a valid provider.
Disable all others providers for embed stream.

This is a fresh install after that steps:
Screen Shot 2019-09-10 at 16 02 13

@cristiandean

This comment has been minimized.

Copy link
Contributor Author

commented Sep 13, 2019

@wyattjoh This happen when we use only one provider #2422 . For this reason, the user don't click in the "Sign in with ..." provider button and the function handleOnClick is not called.

image

The handleOnClick should call redirectOAuth2 that set authRedirectBackTo=/embed/auth in the session storage. The error occurs because the handle click is never called.

@wyattjoh

This comment has been minimized.

Copy link
Member

commented Sep 13, 2019

Great catch! We'll address this in the next release!

wyattjoh added a commit that referenced this issue Sep 13, 2019
wyattjoh added a commit that referenced this issue Sep 17, 2019

@kgardnr kgardnr closed this in 9214610 Sep 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.