Skip to content

Commit

Permalink
fix: Catch Exception within DefaultDelivery class
Browse files Browse the repository at this point in the history
  • Loading branch information
fractalwrench committed Aug 20, 2018
1 parent 30d071d commit f3053ba
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions sdk/src/main/java/com/bugsnag/android/DefaultDelivery.java
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@

class DefaultDelivery implements Delivery {

private static final int HTTP_BAD_REQUEST = 400;
private final ConnectivityManager connectivityManager;

DefaultDelivery(ConnectivityManager connectivityManager) {
Expand Down Expand Up @@ -79,6 +80,9 @@ int deliver(String urlString,
return conn.getResponseCode();
} catch (IOException exception) {
throw new DeliveryFailureException("IOException encountered in request", exception);
} catch (Exception exception) {
Logger.warn("Unexpected error delivering payload", exception);
return HTTP_BAD_REQUEST;
} finally {
IOUtils.close(conn);
}
Expand Down

0 comments on commit f3053ba

Please sign in to comment.