invidious
invidious copied to clipboard
Fix error code for disabled popular endpoint
When visiting /api/v1/popular and popular endpoint is disabled Before:
500 {"error":"Closed stream"}
After
403 {"error":"Administrator has disabled this endpoint."}
This pull request has been automatically marked as stale and will be closed in 30 days because it has not had recent activity and is much likely abandoned or outdated. If you think this pull request is still relevant and applicable, you just have to post a comment and it will be unmarked.
Bump
I don't know what changed, but /api/v1/popular now returns
400 {"error":"Administrator has disabled this endpoint."}
Closing
The PR should probably still be kept to change the HTTP status code from 400 to 403.
The former doesn't exactly fit for a disabled endpoint
The PR should probably still be kept to change the HTTP status code from 400 to 403.
The former doesn't exactly fit for a disabled endpoint
Fair - updated PR
Thanks for the fix!