Adjust logging level for failed OkHttp requests (#6378)

* log level depending on the error type
* Squashing and Merging
* remove unnecessary log (with wrong tag)

1 files changed, 12 insertions, 2 deletions

diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/http/HTTPRequest.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/http/HTTPRequest.java
index a573f7d870..8c818c2f52 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/http/HTTPRequest.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/http/HTTPRequest.java
@@ -138,8 +138,6 @@ class HTTPRequest implements Callback {
     }
 
     private void onFailure(Exception e) {
-        Log.w(LOG_TAG, String.format("[HTTP] Request could not be executed: %s", e.getMessage()));
-
         int type = PERMANENT_ERROR;
         if ((e instanceof NoRouteToHostException) || (e instanceof UnknownHostException) || (e instanceof SocketException) || (e instanceof ProtocolException) || (e instanceof SSLException)) {
             type = CONNECTION_ERROR;
@@ -149,6 +147,18 @@ class HTTPRequest implements Callback {
 
         String errorMessage = e.getMessage() != null ? e.getMessage() : "Error processing the request";
 
+        if (type == TEMPORARY_ERROR) {
+            Log.d(LOG_TAG, String.format(MapboxConstants.MAPBOX_LOCALE,
+                "Request failed due to a temporary error: %s", errorMessage));
+        } else if (type == CONNECTION_ERROR) {
+            Log.i(LOG_TAG, String.format(MapboxConstants.MAPBOX_LOCALE,
+                "Request failed due to a connection error: %s", errorMessage));
+        } else {
+            // PERMANENT_ERROR
+            Log.w(LOG_TAG, String.format(MapboxConstants.MAPBOX_LOCALE,
+                "Request failed due to a permanent error: %s", errorMessage));
+        }
+
         mLock.lock();
         if (mNativePtr != 0) {
             nativeOnFailure(type, errorMessage);