prepare release v1.0.2

jpush · Jan 5, 2017 · ac8ed7f · ac8ed7f
2 parents 253b46c + d20e591
commit ac8ed7f
Show file tree

Hide file tree

Showing 3 changed files with 51 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -57,7 +57,7 @@ public void testSendPushWithCallback() {
 <dependency>
  <groupId>cn.jpush.api</groupId>
  <artifactId>jiguang-common</artifactId>
- <version>1.0.1</version>
+ <version>1.0.2</version>
 </dependency>
 <dependency>
  <groupId>com.google.code.gson</groupId>

diff --git a/pom.xml b/pom.xml
@@ -34,7 +34,7 @@
  <url>https://github.com/jpush/jiguang-java-client-common</url>
  <connection>scm:git:git@github.com:jpush/jiguang-java-client-common.git</connection>
  <developerConnection>scm:git:git@github.com:jpush/jiguang-java-client-common.git</developerConnection>
- <tag>v1.0.1</tag>
+ <tag>v1.0.2</tag>
  </scm>
 
  <dependencies>

diff --git a/src/main/java/cn/jiguang/common/connection/HttpResponseHandler.java b/src/main/java/cn/jiguang/common/connection/HttpResponseHandler.java
@@ -1,6 +1,8 @@
 package cn.jiguang.common.connection;
 
+import cn.jiguang.common.resp.APIRequestException;
 import cn.jiguang.common.resp.ResponseWrapper;
+import io.netty.channel.ChannelHandler;
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.channel.SimpleChannelInboundHandler;
 import io.netty.handler.codec.http.*;
@@ -11,6 +13,7 @@
 import java.util.concurrent.CountDownLatch;
 
 
+@ChannelHandler.Sharable
 public class HttpResponseHandler extends SimpleChannelInboundHandler<HttpObject> {
 
  private static final Logger LOG = LoggerFactory.getLogger(HttpResponseHandler.class);
@@ -41,15 +44,56 @@ protected void channelRead0(ChannelHandlerContext ctx, HttpObject msg) throws Ex
  String responseContent = content.content().toString(CharsetUtil.UTF_8);
  _wrapper.responseCode = status;
  _wrapper.responseContent = responseContent;
- LOG.info("Got Response code: " + status + " content: " + responseContent);
- System.err.println("Got Response code: " + status + " content: " + responseContent);
- System.err.flush();
- if (null != _callback) {
- _callback.onSucceed(_wrapper);
+ if (status >= 200 && status < 300) {
+ LOG.debug("Succeed to get response OK - responseCode:" + status);
+ LOG.debug("Response Content - " + responseContent);
+
+ } else if (status >= 300 && status < 400) {
+ LOG.warn("Normal response but unexpected - responseCode:" + status + ", responseContent:" + responseContent);
+
+ } else {
+ LOG.warn("Got error response - responseCode:" + status + ", responseContent:" + responseContent);
+
+ switch (status) {
+ case 400:
+ LOG.error("Your request params is invalid. Please check them according to error message.");
+ _wrapper.setErrorObject();
+ break;
+ case 401:
+ LOG.error("Authentication failed! Please check authentication params according to docs.");
+ _wrapper.setErrorObject();
+ break;
+ case 403:
+ LOG.error("Request is forbidden! Maybe your appkey is listed in blacklist or your params is invalid.");
+ _wrapper.setErrorObject();
+ break;
+ case 404:
+ LOG.error("Request page is not found! Maybe your params is invalid.");
+ _wrapper.setErrorObject();
+ break;
+ case 410:
+ LOG.error("Request resource is no longer in service. Please according to notice on official website.");
+ _wrapper.setErrorObject();
+ case 429:
+ LOG.error("Too many requests! Please review your appkey's request quota.");
+ _wrapper.setErrorObject();
+ break;
+ case 500:
+ case 502:
+ case 503:
+ case 504:
+ LOG.error("Seems encountered server error. Maybe JPush is in maintenance? Please retry later.");
+ break;
+ default:
+ LOG.error("Unexpected response.");
+ }
  }
  if (null != _latch) {
  _latch.countDown();
  }
+ if (null != _callback) {
+ _callback.onSucceed(_wrapper);
+ }
  }
  }
  }