Better logging in case MPS response codes does not match.

Author: hirro

pom.xml

@@ -123,7 +123,7 @@
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-api</artifactId>
-            <version>1.6.1</version>
+            <version>1.7.7</version>
         </dependency>
 
         <dependency>

src/main/java/com/notnoop/mpns/internal/Utilities.java

@@ -40,6 +40,7 @@
 import org.apache.http.Header;
 import org.apache.http.HttpResponse;
 import org.apache.http.impl.conn.tsccm.ThreadSafeClientConnManager;
+import org.slf4j.LoggerFactory;
 
 public final class Utilities {
     private Utilities() { throw new AssertionError("Uninstantiable class"); }
@@ -49,6 +50,8 @@ public final class Utilities {
      */
     public static String XML_CONTENT_TYPE = "text/xml";
 
+    private static final org.slf4j.Logger LOG = LoggerFactory.getLogger(Utilities.class);
+
     public static ThreadSafeClientConnManager poolManager(int maxConnections) {
         ThreadSafeClientConnManager cm = new ThreadSafeClientConnManager();
         cm.setMaxTotal(maxConnections);
@@ -158,8 +161,13 @@ public static MpnsResponse logicalResponseFor(HttpResponse response) {
         }
 
         // Didn't find anything
-        assert false;
-        return null;
+        LOG.error(
+                "Unmatched error code - Notification status: [{}], Connection status: [{}], Subscription status: [{}], Status code: [{}]",
+                notificationStatus,
+                deviceConnectionStatus,
+                subscriptionStatus,
+                statusCode);
+        return MpnsResponse.UNDEFINED;
     }
 
     public static void fireDelegate(MpnsNotification message, HttpResponse response, MpnsDelegate delegate) {