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

Discord login does not work just after logout #1454

Closed
2 of 3 tasks
dyc3 opened this issue Mar 6, 2024 · 0 comments · Fixed by #1457
Closed
2 of 3 tasks

Discord login does not work just after logout #1454

dyc3 opened this issue Mar 6, 2024 · 0 comments · Fixed by #1457
Labels
bug Something isn't working regression An intentional behavior that worked before does not work now

Comments

@dyc3
Copy link
Owner

dyc3 commented Mar 6, 2024 •
edited
Loading

Current Behavior

No response

Expected Behavior

discord logins should always work

Steps To Reproduce

  1. Log in with discord
  2. Log out
  3. try to log in with discord again
  4. see error

Environment

  • This happens on the official site, opentogethertube.com
  • This happens using a self-hosted version.
  • I'm using the docker image.

Anything else?

No response
@dyc3 dyc3 added bug Something isn't working regression An intentional behavior that worked before does not work now labels Mar 6, 2024
@dyc3 dyc3 changed the title Discord logins are broken Discord login does not work just after logout Mar 6, 2024
dyc3 added a commit that referenced this issue Mar 6, 2024
@dyc3
put auth token in cookies
f4701cf 
fixes #1454
helps with #1456
@dyc3 dyc3 mentioned this issue Mar 6, 2024
Merged
dyc3 added a commit that referenced this issue Mar 6, 2024
@dyc3
put auth token in cookies
177afcd 
fixes #1454
helps with #1456
dyc3 added a commit that referenced this issue Mar 6, 2024
@dyc3
put auth token in cookies
fe9012d 
fixes #1454
helps with #1456
dyc3 added a commit that referenced this issue Mar 7, 2024
@dyc3
put auth token in cookies
d642cd0 
fixes #1454
helps with #1456
@dyc3 dyc3 closed this as completed in #1457 Mar 7, 2024
dyc3 added a commit that referenced this issue Mar 7, 2024
@dyc3
put auth token in cookies (#1457)
cc043a5 
* put auth token in cookies

fixes #1454
helps with #1456

* remove lusca because csrf is mitigated by cookie options
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working regression An intentional behavior that worked before does not work now
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

put auth token in cookies dyc3/opentogethertube
1 participant
@dyc3