Commitd7ab7ec

authored and

committed

Added better handling of communication errors

(cherry picked from commit2893283)

1 parent1e03995 commitd7ab7ecCopy full SHA for d7ab7ec

File tree

-2

lines changed

src/main/java/io/api/etherscan
- error
  - ConnectionException.java
- executor/impl
  - HttpExecutor.java

-2

lines changed

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,10 @@`
`8`	`8`	`*/`
`9`	`9`	`publicclassConnectionExceptionextendsApiException {`
`10`	`10`
	`11`	`+publicConnectionException(Stringmessage) {`
	`12`	`+super(message);`
	`13`	`+ }`
	`14`	`+`
`11`	`15`	`publicConnectionException(Stringmessage,Throwablecause) {`
`12`	`16`	`super(message,cause);`
`13`	`17`	`}`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -18,8 +18,7 @@`
`18`	`18`	`importjava.util.zip.GZIPInputStream;`
`19`	`19`	`importjava.util.zip.InflaterInputStream;`
`20`	`20`
`21`		`-importstaticjava.net.HttpURLConnection.HTTP_MOVED_PERM;`
`22`		`-importstaticjava.net.HttpURLConnection.HTTP_MOVED_TEMP;`
	`21`	`+importstaticjava.net.HttpURLConnection.*;`
`23`	`22`
`24`	`23`	`/**`
`25`	`24`	`* Http client implementation`
`@@ -88,6 +87,10 @@ public String get(final String urlAsString) {`
`88`	`87`	`finalintstatus =connection.getResponseCode();`
`89`	`88`	`if (status ==HTTP_MOVED_TEMP \|\|status ==HTTP_MOVED_PERM) {`
`90`	`89`	`returnget(connection.getHeaderField("Location"));`
	`90`	`+ }elseif ((status >=HTTP_BAD_REQUEST) && (status <HTTP_INTERNAL_ERROR)) {`
	`91`	`+thrownewConnectionException("Protocol error: "+connection.getResponseMessage());`
	`92`	`+ }elseif (status >=HTTP_INTERNAL_ERROR) {`
	`93`	`+thrownewConnectionException("Server error: "+connection.getResponseMessage());`
`91`	`94`	`}`
`92`	`95`
`93`	`96`	`finalStringdata =readData(connection);`

Comments

(0)