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.

Sign up for GitHub

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

Jump to bottom

Small improvement: remove unnecessary login redirect #285

Closed
2 of 4 tasks
msm-code opened this issue Sep 6, 2022 · 0 comments · Fixed by #322
Closed
2 of 4 tasks

Small improvement: remove unnecessary login redirect #285

msm-code opened this issue Sep 6, 2022 · 0 comments · Fixed by #322
Labels
priority:low Priority: low type:feature Add a new feature
Milestone
v1.4.0

Comments

@msm-code
Copy link
Contributor

msm-code commented Sep 6, 2022

Feature Category

  • Correctness
  • User Interface / User Experience
  • Performance
  • Other (please explain)

Describe the problem

When logging in, clicking the "login" button redirects to /auth and then immediately to the correct URL. This causes unnecessary screen flickering and bad UX.

Describe the solution you'd like

Just redirect directly to OIDC login url.

Describe alternatives you've considered

No reason to keep the current behaviour. Right now redirect to /auth is a nice trick to force login, but I don't think this is a very useful feature.
@msm-code msm-code added priority:low Priority: low type:feature Add a new feature labels Sep 6, 2022
@msm-code msm-code mentioned this issue Jan 11, 2023
Merged
4 tasks
@msm-code msm-code added this to the v1.4.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority:low Priority: low type:feature Add a new feature
Projects
None yet
Milestone
v1.4.0
Development

Successfully merging a pull request may close this issue.

Make login faster - no unnecessary redirects CERT-Polska/mquery
1 participant
@msm-code