Skip to content

phabricator: Catch JSONDecodeError

BryanDavis requested to merge work/bd808/phab-response into main

Error logging shows that Phabricator can occasionally return a malformed response that is not valid JSON. By converting the JSONDecodeError to an HTTPError we give the requests library a chance to retry the query if that is still possible by config. If returns are exhausted we will still give a bit of a better clue about what went wrong than just letting the JSONDecodeError rip up the stack.

Merge request reports

Loading