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

Support for Safari #184

Closed
jaredkipe opened this issue Jun 27, 2020 · 2 comments · Fixed by #185
Closed

Support for Safari #184

jaredkipe opened this issue Jun 27, 2020 · 2 comments · Fixed by #185
Assignees
Labels
bug this needs to be fixed UI User Interface

Comments

@jaredkipe
Copy link

In Safari, Login page works for getting errors when credentials are wrong, but when you provide correct credentials the page reloads and you do not get any visible errors anywhere.

the 'token' cookie does get set, and it does have the correct domain/content.
there is no local storage created

FF has both after login.

@dvaldivia
Copy link
Collaborator

nice catch @jaredkipe I see the behavior, strangely enough, on local it doesn't happen, only when deployed, we'll investigate.

@dvaldivia dvaldivia added bug this needs to be fixed UI User Interface labels Jun 29, 2020
@dvaldivia
Copy link
Collaborator

I see that the cookie is getting attached on localhost but not when deployed online, however we are actually passing the Bearer authorization header, so it shouldn't matter, maybe the missing cookie has something to do with privacy setting in Safari

On Local
Screen Shot 2020-06-29 at 10 05 40 AM

Deployed (play)
Screen Shot 2020-06-29 at 10 05 47 AM

@dvaldivia dvaldivia assigned Alevsk and unassigned dvaldivia and bexsoft Jun 29, 2020
Alevsk added a commit to Alevsk/console that referenced this issue Jun 30, 2020
fixes: minio#184

There was a bug in Safari in related to the browser not setting the session token
correctly in localstorage, this was because we were using
window.location.href for redirect instead of history.push after login, the redirect execution was faster
was faster that the promise function getting the response after the login request
and it seems to be that Safari will kill all current request of a
window when the page is getting redirected.

Test this:

Try to sign-in using Safari browser (latest version is recommended)
Alevsk added a commit to Alevsk/console that referenced this issue Jun 30, 2020
fixes: minio#184

There was a bug in Safari in related to the browser not setting the session token
correctly in localstorage, this was because we were using
window.location.href for redirect instead of history.push after login, the redirect execution was faster
was faster that the promise function getting the response after the login request
and it seems to be that Safari will kill all current request of a
window when the page is getting redirected.

Test this:

Try to sign-in using Safari browser (latest version is recommended)
Alevsk added a commit to Alevsk/console that referenced this issue Jun 30, 2020
fixes: minio#184

There was a bug in Safari in related to the browser not setting the session token
correctly in localstorage, this was because we were using
window.location.href for redirect instead of history.push after login, the redirect execution was faster
was faster that the promise function getting the response after the login request
and it seems to be that Safari will kill all current request of a
window when the page is getting redirected.

Test this:

Try to sign-in using Safari browser (latest version is recommended)
Alevsk added a commit that referenced this issue Jun 30, 2020
fixes: #184

There was a bug in Safari in related to the browser not setting the session token
correctly in localstorage, this was because we were using
window.location.href for redirect instead of history.push after login, the redirect execution was faster
was faster that the promise function getting the response after the login request
and it seems to be that Safari will kill all current request of a
window when the page is getting redirected.

Test this:

Try to sign-in using Safari browser (latest version is recommended)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug this needs to be fixed UI User Interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants