summaryrefslogtreecommitdiffstats
path: root/python/fatcat_tools/importers/common.py
diff options
context:
space:
mode:
authorBryan Newbold <bnewbold@robocracy.org>2020-03-25 13:28:31 -0700
committerBryan Newbold <bnewbold@robocracy.org>2020-03-25 13:28:33 -0700
commitb9657b39f4061e4cf870fa253c5665ab5ad60cc6 (patch)
tree8aab666677ba4bc86992a874ce0360e22e06dfb8 /python/fatcat_tools/importers/common.py
parent678a2ca17a167f7893173664bad22c5c4c0181a0 (diff)
downloadfatcat-b9657b39f4061e4cf870fa253c5665ab5ad60cc6.tar.gz
fatcat-b9657b39f4061e4cf870fa253c5665ab5ad60cc6.zip
catch ApiValueError in some generic API calls
The motivation for this change is to handle bogus revision IDs in URLs, which were causing 500 errors not 400 errors. Eg: https://qa.fatcat.wiki/file/rev/5d5d5162-b676-4f0a-968f-e19dadeaf96e%2B2019-11-27%2B13:49:51%2B0%2B6 I have no idea where these URLs are actually coming from, but they should be 4xx not 5xx. Investigating made me realize there is a whole category of ApiValueError exceptions we were not catching and should have been.
Diffstat (limited to 'python/fatcat_tools/importers/common.py')
0 files changed, 0 insertions, 0 deletions