osu-tournament-rating / otr-api

API powering osu! Tournament Rating
https://otr.stagec.xyz/
GNU General Public License v3.0
8 stars 5 forks source link

Fix and improve API rate limiting #514

Closed myssto closed 1 month ago

myssto commented 1 month ago

Funnily enough, I don't think the rate limiter was ever working as intended to begin with? After debugging for a little it seemed like the way we were parsing the claims principal for partitioning (through the IAuthenticateResultFeature) was either just not working at all, or incredibly inconsistent. Either way it's good now :D

I use the royal we as if I wasn't the one who wrote all of this in the first place

github-actions[bot] commented 1 month ago

Qodana for .NET

1 new problem were found

Inspection name Severity Problems
Use preferred style for trailing comma before new line in multiline lists ◽️ Notice 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked ☁️ View the detailed Qodana report

Contact Qodana team Contact us at [qodana-support@jetbrains.com](mailto:qodana-support@jetbrains.com) - Or via our issue tracker: https://jb.gg/qodana-issue - Or share your feedback: https://jb.gg/qodana-discussions
github-actions[bot] commented 1 month ago

Qodana for .NET

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked ☁️ View the detailed Qodana report

Contact Qodana team Contact us at [qodana-support@jetbrains.com](mailto:qodana-support@jetbrains.com) - Or via our issue tracker: https://jb.gg/qodana-issue - Or share your feedback: https://jb.gg/qodana-discussions