Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Unhandled value out of range #827

Closed
krysal opened this issue Jul 28, 2022 · 0 comments · Fixed by #903
Closed

Unhandled value out of range #827

krysal opened this issue Jul 28, 2022 · 0 comments · Fixed by #903
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟨 priority: medium Not blocking but should be addressed soon 🐛 tooling: sentry Sentry issue

Comments

@krysal
Copy link
Member

krysal commented Jul 28, 2022

Sentry link

https://sentry.io/share/issue/7dd836d27daa41dfa59dfbdf39a40d51/

Description

We need to capture and handle this error to avoid returning a 500 error in the link below.

'l' format requires -2147483648 <= number <= 2147483647

Originates from this fragment of code:

if exif:
exif_bytes = piexif.dump(exif)
else:
exif_bytes = None
img_bytes = io.BytesIO()
self._save_wrapper(watermarked, exif_bytes, img_bytes)

Reproduction

  1. Go to https://api-dev.openverse.engineering/v1/images/6c855680-b426-40f1-adc6-f61f434d0be9/watermark/

Aditional context

Related to #483.

@krysal krysal added 🟨 priority: medium Not blocking but should be addressed soon 🛠 goal: fix Bug fix 💻 aspect: code Concerns the software code in the repository 🐛 tooling: sentry Sentry issue labels Jul 28, 2022
@krysal krysal changed the title <Replace this with actual title> Unhandled value out of range Jul 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟨 priority: medium Not blocking but should be addressed soon 🐛 tooling: sentry Sentry issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant