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

Set defaults for legacy IO to match osu web development environment #270

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

peppy
Copy link
Member

@peppy peppy commented Feb 12, 2025

Note that empty string for the secret here does match and work with a default osu-web deploy.

@smoogipoo
Copy link
Contributor

Ouch, those exceptions wouldn't have fired properly. Might as well reference the static vars if you're going to remove them though?

@smoogipoo smoogipoo merged commit a727105 into ppy:master Feb 12, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants