-
Notifications
You must be signed in to change notification settings - Fork 63
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
Check for lobby scope in token during login #840
Conversation
Hmmm not sure why that test is failing because it uses the old password login |
When run by itself it passes just fine |
Codecov Report
|
Just a flakey test |
Also this is super important for enabling Gog link 🚀 wheeeeee |
Oh wait. This PR should instead remove the check for Steam link as this moves to the user service. |
The steam link check only happens when you login with password |
So I'm assuming when you try to log in via the API and you're banned, you see the ban reason on the webpage? |
Correct |
f574088
to
396849a
Compare
This asserts that the oauth token contains the lobby scope before allowing a player to login. This allows for players to be banned from just the game service. All oauth tokens from the clients currently have the lobby scope