Skip to content

Front end shall not show login page if session is logged in #1499

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

Closed
pcrespov opened this issue May 12, 2020 · 0 comments · Fixed by #1516
Closed

Front end shall not show login page if session is logged in #1499

pcrespov opened this issue May 12, 2020 · 0 comments · Fixed by #1516
Assignees
Labels
a:frontend issue affecting the front-end (area group) t:enhancement Improvement or request on an existing feature

Comments

@pcrespov
Copy link
Member

  1. set WEBSERVER_LOGIN_REGISTRATION_CONFIRMATION_REQUIRED=0 and WEBSERVER_LOGIN_REGISTRATION_INVITATION_REQUIRED=0
  2. start osparc
  3. Click create account
  4. add user and password x2. Press Submit
    Screen Shot 2020-05-12 at 18.21.56.png
    -> goes back to login page but a blue pop-up info message saying "You are logged in". So, should I introduce my password??
    Screen Shot 2020-05-12 at 18.25.27.png

It reality a cookie with a session was already provided so the user SHOULD NOT be prompted to introduce the user/password. Just by reloading the page one can see that it front-end now shows the dashboard

@pcrespov pcrespov added a:frontend issue affecting the front-end (area group) bug buggy, it does not work as expected labels May 12, 2020
@ignapas ignapas added t:enhancement Improvement or request on an existing feature and removed bug buggy, it does not work as expected labels May 20, 2020
@ignapas ignapas mentioned this issue May 20, 2020
1 task
@ignapas ignapas linked a pull request May 20, 2020 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:frontend issue affecting the front-end (area group) t:enhancement Improvement or request on an existing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants