-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
unit3d: move api key auth in headers #14429
Conversation
Maybe in the future... |
No issue with this, we already use API key headers for PolishTracker and Milkie. |
I'll revise in the future. 👍 |
ab6e67a
to
9f25023
Compare
9f25023
to
e8dc35f
Compare
e8dc35f
to
babe693
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working well.
Just need a few others tested:
- blutopia
- hd-unit3d
- r4e
- redbits
- skipthetrailers
- theoldschool
- theshinning
There's also a couple down and a few that no active maintainers have accounts for, but I doubt they'll be an issue.
Since you merged this PR, HD-Olimpo (API) - https://hd-olimpo.club/ began to fails it's uses unit3d too |
Thanks, fixed in dcec234. |
Description
Moving the api keys to headers, thus making the URLs with credentials another thing not to worry about.