Page MenuHomePhabricator

Properly handle malformed curl results
Closed, ResolvedPublic1 Estimated Story Points

Description

Currently, if a curl result is bad, it returns a 500 error. We should be able to handle these errors in all instances, and return default values. This may require try/catch blocks.

Event Timeline

Matthewrbowker triaged this task as Low priority.
Matthewrbowker moved this task from General / other to Working on the XTools board.
Matthewrbowker moved this task from Inbox to Working on the User-Matthewrbowker board.
Matthewrbowker moved this task from Working to Pending deployment on the XTools board.
Matthewrbowker set the point value for this task to 1.

I'm going to close this now, I made a few changes within API helper to catch exceptions based on "\Exception" - e.g. https://github.com/x-tools/xtools/blob/rfa/src/AppBundle/Helper/ApiHelper.php#L345 This is working in my testing.

This will be merged when https://github.com/x-tools/xtools/pull/67 is merged. Any additional errors can re-open this task.