Merge pull request #52 from chrisdinn/issue_51

Yamil Asusta · Yamil Asusta · commit cb63d823019e · 2014-11-06T15:46:16.000-03:00
Throw SendGridException on IOException communicating with SendGrid API
diff --git a/src/main/java/com/sendgrid/SendGrid.java b/src/main/java/com/sendgrid/SendGrid.java
@@ -148,7 +148,7 @@ public SendGrid.Response send(Email email) throws SendGridException {
       HttpResponse res = this.client.execute(httppost);
       return new SendGrid.Response(res.getStatusLine().getStatusCode(), EntityUtils.toString(res.getEntity()));
     } catch (IOException e) {
-      return new SendGrid.Response(500, "Problem connecting to SendGrid");
+      throw new SendGridException(e);
     }
 
   }

Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,7 @@ public SendGrid.Response send(Email email) throws SendGridException {`
`148`	`148`	`HttpResponse res = this.client.execute(httppost);`
`149`	`149`	`return new SendGrid.Response(res.getStatusLine().getStatusCode(), EntityUtils.toString(res.getEntity()));`
`150`	`150`	`} catch (IOException e) {`
`151`		`- return new SendGrid.Response(500, "Problem connecting to SendGrid");`
	`151`	`+ throw new SendGridException(e);`
`152`	`152`	`}`
`153`	`153`
`154`	`154`	`}`