4xx generated empty responses
This commit is contained in:
@@ -37,7 +37,7 @@ RETURNS cdb_dataservices_server.simple_route AS $$
|
|||||||
|
|
||||||
resp = client.calculate_route_point_to_point(origin_coordinates, dest_coordinates, mode, options, units)
|
resp = client.calculate_route_point_to_point(origin_coordinates, dest_coordinates, mode, options, units)
|
||||||
|
|
||||||
if resp:
|
if resp and resp.shape:
|
||||||
shape_linestring = polyline_to_linestring(resp.shape)
|
shape_linestring = polyline_to_linestring(resp.shape)
|
||||||
quota_service.increment_success_service_use()
|
quota_service.increment_success_service_use()
|
||||||
return [shape_linestring, resp.length, resp.duration]
|
return [shape_linestring, resp.length, resp.duration]
|
||||||
|
|||||||
@@ -45,6 +45,8 @@ class MapzenRouting:
|
|||||||
response = requests.get(self._url, params=request_params)
|
response = requests.get(self._url, params=request_params)
|
||||||
if response.status_code == requests.codes.ok:
|
if response.status_code == requests.codes.ok:
|
||||||
return self.__parse_routing_response(response.text)
|
return self.__parse_routing_response(response.text)
|
||||||
|
elif response.status_code == requests.codes.bad_request:
|
||||||
|
return MapzenRoutingResponse(None, None, None)
|
||||||
else:
|
else:
|
||||||
response.raise_for_status()
|
response.raise_for_status()
|
||||||
|
|
||||||
@@ -78,9 +80,7 @@ class MapzenRouting:
|
|||||||
shape = PolyLine().decode(legs['shape'])
|
shape = PolyLine().decode(legs['shape'])
|
||||||
length = legs['summary']['length']
|
length = legs['summary']['length']
|
||||||
duration = legs['summary']['time']
|
duration = legs['summary']['time']
|
||||||
routing_response = MapzenRoutingResponse(shape, length, duration)
|
return MapzenRoutingResponse(shape, length, duration)
|
||||||
|
|
||||||
return routing_response
|
|
||||||
except IndexError:
|
except IndexError:
|
||||||
return []
|
return []
|
||||||
except KeyError:
|
except KeyError:
|
||||||
|
|||||||
Reference in New Issue
Block a user