Merge pull request #37 from imsweb/update-okhttp

Update okhttp

Author: ctmay4

build.gradle

@@ -39,6 +39,9 @@ dependencies {
     api 'com.squareup.retrofit2:retrofit:2.9.0'
     api 'com.squareup.retrofit2:converter-jackson:2.9.0'
 
+    // retrofit will not update these dependencies to fix vulnerabilities
+    api 'com.squareup.okhttp3:okhttp:4.9.3'
+
     // newer version of dependency to fix vulnerability until converter-jackson is updated
     api 'com.fasterxml.jackson.core:jackson-databind:2.13.3'
 

src/main/java/com/imsweb/seerapi/client/ErrorInterceptor.java

@@ -5,26 +5,32 @@
 
 import java.io.IOException;
 
+import org.jetbrains.annotations.NotNull;
+
 import com.fasterxml.jackson.databind.ObjectMapper;
 
 import okhttp3.Interceptor;
 import okhttp3.Response;
+import okhttp3.ResponseBody;
 
 /**
  * Interceptor to catch all non-200 responses and convert them to exceptions.
  */
 public class ErrorInterceptor implements Interceptor {
 
+    @NotNull
     @Override
     public Response intercept(Chain chain) throws IOException {
         Response response = chain.proceed(chain.request());
 
         if (response.code() != 200) {
-            // convert body to error response
             ErrorResponse error = null;
-            if (response.body() != null) {
+
+            // convert body to error response
+            ResponseBody body = response.body();
+            if (body != null) {
                 try {
-                    error = new ObjectMapper().readValue(response.body().byteStream(), ErrorResponse.class);
+                    error = new ObjectMapper().readValue(body.byteStream(), ErrorResponse.class);
                 }
                 catch (IOException e) {
                     // sometimes the error message is not right format (like for 404 errors)