Skip to content

fix: storage.get return None instead of raise the NoSuchKey error#161

Merged
Bladrak merged 1 commit intothumbor-community:masterfrom
brambaud:fix/storage-get-error-when-image-does-not-exist
Aug 18, 2025
Merged

fix: storage.get return None instead of raise the NoSuchKey error#161
Bladrak merged 1 commit intothumbor-community:masterfrom
brambaud:fix/storage-get-error-when-image-does-not-exist

Conversation

@brambaud
Copy link

Tested Thumbor 7.7.4, when STORAGE = 'tc_aws.storages.s3_storage' it crashes due to a botocore.errorfactory.NoSuchKey error.

Also become a little bit more consistent with result storage's get method which does the same.

@brambaud
Copy link
Author

Pipeline is failing because it takes too long to install the dependencies
Not due to the changes of this PR

@Bladrak
Copy link

Bladrak commented Aug 13, 2025

@brambaud thanks, if you can rebase CI should be fixed

Tested Thumbor 7.7.4, when `STORAGE = 'tc_aws.storages.s3_storage'` it crashes due to a botocore.errorfactory.NoSuchKey error.

Also become a little bit more consistent with result storage's get method which does the same.
@brambaud brambaud force-pushed the fix/storage-get-error-when-image-does-not-exist branch from daa8ca5 to 9052f79 Compare August 14, 2025 09:56
@brambaud
Copy link
Author

Done

@Bladrak Bladrak merged commit ffebd8e into thumbor-community:master Aug 18, 2025
2 checks passed
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