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

Added several retries when uploading thumbnail #160

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

benoit-leveau
Copy link

Code is quite basic but shows what should happen in a few places in toolkit, with proper error reporting of the server response.

Also, we usually add an extra error reporting to pipeline by email as this could indicate a serious error and we want to be informed as soon as possible (artists might not always report errors to us).
This would be good to have a general system for error reporting to pipeline, like a hook called whenever an error occurs. As dissussed in the catchup I tried to add this directly in TankError, to catastrophic results (a few thousand emails were sent just by launching an app because a TankError is raised by every template when trying to match any path).

@jfboismenu
Copy link
Contributor

Hi Benoit, is this still a problem or has Shotgun become more reliable when uploading media?

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