-
Notifications
You must be signed in to change notification settings - Fork 50
Conversation
API Developer Docs Preview: Ready https://wordpress.github.io/openverse-api/_preview/756 Please note that GitHub pages takes a little time to deploy newly pushed code, if the links above don't work or you see old versions, wait 5 minutes and try again. You can check the GitHub pages deployment action list to see the current status of the deployments. |
raise get_api_exception(f"Failed to render thumbnail: {exc}") | ||
except Exception as exc: | ||
raise get_api_exception( |
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.
Nice for debugging in the future :)
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.
I tried getting 10 thumbnails using a python script, and all returned 200 :)
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.
Looks great, thank you!
Fixes
Fixes #695 by @AetherUnbound
Description
This PR prevents thumbnail errors from causing server crashes by
Testing Instructions
Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin