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

[bug]: cancelling in GOOGLE LOGIN should forward to "/login" #2585

Open
bandhan-majumder opened this issue Oct 12, 2024 · 5 comments · May be fixed by #2659
Open

[bug]: cancelling in GOOGLE LOGIN should forward to "/login" #2585

bandhan-majumder opened this issue Oct 12, 2024 · 5 comments · May be fixed by #2659
Assignees

Comments

@bandhan-majumder
Copy link

Describe the bug
While denying the access permission in google based auth, the error is forwarding to route https://app.tolgee.io/login/auth_callback/google?error=access_denied which is not opening. It should forward the user to the home page.

To Reproduce
Steps to reproduce the behavior:

  1. Go to here
  2. Click on GOOGLE LOGIN
  3. Choose account
  4. Click on cancel
  5. See the error with the route it is forwarding to

Expected behavior
If there is an error, it should move the user to "/login" route

Screenshots
image

@bandhan-majumder
Copy link
Author

Can this issue be approved with "hacktoberfest" label? I would like to self assign this issue.

@ritikk112
Copy link

I would like to work on this, can you assign me this issue?

@bandhan-majumder
Copy link
Author

Sorry for being so late and not being able to solve the issue. It anyone else wants to take the issue, please take it. I am unassigning myself. Thanks for the opportunity. In future I hope to solve issues in this repo.

@bandhan-majumder bandhan-majumder removed their assignment Oct 24, 2024
@krushnarout
Copy link

Hi @JanCizmar, I have resolved the issue locally. Could you please assign me so I can proceed with making a pull request?

@JanCizmar
Copy link
Contributor

@krushnarout You don't have to be assigned to create PR, but why not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants