[Feat] [SDK-399] Okhttp interceptor

rollbar-okhttp/README.md

@@ -0,0 +1,75 @@
+# Rollbar OkHttp Integration
+
+This module provides an [OkHttp Interceptor](https://square.github.io/okhttp/features/interceptors/) that automatically captures network telemetry for the Rollbar Java SDK.
+
+It records:
+
+- **Network telemetry events** for HTTP responses with status code `>= 400` (client and server errors).
+- **Error events** for connection failures, timeouts, and other I/O exceptions.
+
+## Installation
+
+### Gradle (Kotlin DSL)
+
+```kotlin
+dependencies {
+    implementation("com.rollbar:rollbar-okhttp:<version>")
+    implementation("com.squareup.okhttp3:okhttp:<okhttp-version>")
+}
+```
+
+### Gradle (Groovy)
+
+```groovy
+dependencies {
+    implementation 'com.rollbar:rollbar-okhttp:<version>'
+    implementation 'com.squareup.okhttp3:okhttp:<okhttp-version>'
+}
+```
+
+## Usage
+
+### 1. Implement `NetworkTelemetryRecorder`
+
+```java
+NetworkTelemetryRecorder recorder = new NetworkTelemetryRecorder() {
+    @Override
+    public void recordNetworkEvent(Level level, String method, String url, String statusCode) {
+        rollbar.recordNetworkEventFor(level, method, url, statusCode);
+    }
+
+    @Override
+    public void recordErrorEvent(Exception exception) {
+        rollbar.log(exception);
+    }
+};
+```
+
+### 2. Add the interceptor to your OkHttpClient
+
+```java
+OkHttpClient client = new OkHttpClient.Builder()
+    .addInterceptor(new RollbarOkHttpInterceptor(recorder))
+    .build();
+```
+
+### 3. Make requests as usual
+
+```java
+Request request = new Request.Builder()
+    .url("https://api.example.com/data")
+    .build();
+
+Response response = client.newCall(request).execute();
+```
+
+The interceptor will automatically record telemetry events to Rollbar without interfering with the request/response flow.
+
+## Behavior
+
+| Scenario                          | Action                                                  |
+|-----------------------------------|---------------------------------------------------------|
+| Recorder is `null`                | No telemetry or log is recorded                         |
+| Response status `< 400`           | No telemetry recorded, response returned normally       |
+| Response status `>= 400`          | Records a network telemetry event with `Level.CRITICAL` |
+| Connection failure / timeout      | Records an error event, then rethrows the `IOException` |

rollbar-okhttp/build.gradle.kts

@@ -0,0 +1,23 @@
+plugins {
+    id("java")
+}
+
+group = "com.rollbar.okhttp"
+
+repositories {
+    mavenCentral()
+}
+
+dependencies {
+    testImplementation(platform("org.junit:junit-bom:5.14.3"))
+    testImplementation("org.junit.jupiter:junit-jupiter")
+    testRuntimeOnly("org.junit.platform:junit-platform-launcher")
+    testImplementation("com.squareup.okhttp3:mockwebserver:5.3.2")
+    testImplementation("org.mockito:mockito-core:5.23.0")
+    implementation("com.squareup.okhttp3:okhttp:5.3.2")
+    api(project(":rollbar-api"))
+}
+
+tasks.test {
+    useJUnitPlatform()
+}

rollbar-okhttp/src/main/java/com/rollbar/okhttp/NetworkTelemetryRecorder.java

@@ -0,0 +1,9 @@
+package com.rollbar.okhttp;
+
+import com.rollbar.api.payload.data.Level;
+
+public interface NetworkTelemetryRecorder {
+  void recordNetworkEvent(Level level, String method, String url, String statusCode);
+
+  void recordErrorEvent(Exception exception);
+}

rollbar-okhttp/src/main/java/com/rollbar/okhttp/RollbarOkHttpInterceptor.java

@@ -0,0 +1,61 @@
+package com.rollbar.okhttp;
+
+import com.rollbar.api.payload.data.Level;
+
+import java.io.IOException;
+import java.util.logging.Logger;
+
+import okhttp3.Interceptor;
+import okhttp3.Request;
+import okhttp3.Response;
+
+public class RollbarOkHttpInterceptor implements Interceptor {
+
+  private static final Logger LOGGER = Logger.getLogger(RollbarOkHttpInterceptor.class.getName());
+
+  private final NetworkTelemetryRecorder recorder;
+
+  public RollbarOkHttpInterceptor(NetworkTelemetryRecorder recorder) {
+    this.recorder = recorder;
+  }
+
+  @Override
+  public Response intercept(Chain chain) throws IOException {
+    Request request = chain.request();
+
+    try {
+      Response response = chain.proceed(request);
+
+      if (response.code() >= 400 && recorder != null) {
+        try {
+          recorder.recordNetworkEvent(
+              Level.CRITICAL,
+              request.method(),
+              request.url().toString(),
+              String.valueOf(response.code()));
+        } catch (Exception recorderException) {
+          LOGGER.log(java.util.logging.Level.WARNING,
+              "NetworkTelemetryRecorder.recordNetworkEvent threw an exception; "
+                  + "suppressing to preserve the interceptor contract.",
+              recorderException);
+        }
+      }
+
+      return response;
+
+    } catch (IOException e) {
+      if (recorder != null) {
+        try {
+          recorder.recordErrorEvent(e);
+        } catch (Exception recorderException) {
+          LOGGER.log(java.util.logging.Level.WARNING,
+              "NetworkTelemetryRecorder.recordErrorEvent threw an exception; "
+                  + "suppressing to preserve the original IOException.",
+              recorderException);
+        }
+      }
+
+      throw e;
+    }
+  }
+}

rollbar-okhttp/src/test/java/com/rollbar/okhttp/RollbarOkHttpInterceptorTest.java

@@ -0,0 +1,177 @@
+package com.rollbar.okhttp;
+
+import com.rollbar.api.payload.data.Level;
+import okhttp3.OkHttpClient;
+import okhttp3.Request;
+import okhttp3.Response;
+import okhttp3.mockwebserver.MockResponse;
+import okhttp3.mockwebserver.MockWebServer;
+import okhttp3.mockwebserver.SocketPolicy;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+import java.io.IOException;
+
+import static org.junit.jupiter.api.Assertions.*;
+import static org.mockito.Mockito.*;
+
+class RollbarOkHttpInterceptorTest {
+
+    private MockWebServer server;
+    private NetworkTelemetryRecorder recorder;
+    private OkHttpClient client;
+
+    @BeforeEach
+    void setUp() throws IOException {
+        server = new MockWebServer();
+        server.start();
+
+        recorder = mock(NetworkTelemetryRecorder.class);
+
+        client = new OkHttpClient.Builder()
+                .addInterceptor(new RollbarOkHttpInterceptor(recorder))
+                .build();
+    }
+
+    @AfterEach
+    void tearDown() throws IOException {
+        server.shutdown();
+    }
+
+    @Test
+    void successfulResponse_doesNotRecordEvent() throws IOException {
+        server.enqueue(new MockResponse().setResponseCode(200));
+
+        Request request = new Request.Builder().url(server.url("/ok")).build();
+        Response response = client.newCall(request).execute();
+        response.close();
+
+        assertEquals(200, response.code());
+        verifyNoInteractions(recorder);
+    }
+
+    @Test
+    void redirectResponse_doesNotRecordEvent() throws IOException {
+        server.enqueue(new MockResponse().setResponseCode(301).addHeader("Location", "/other"));
+
+        OkHttpClient noFollowClient = client.newBuilder().followRedirects(false).build();
+        Request request = new Request.Builder().url(server.url("/redirect")).build();
+        Response response = noFollowClient.newCall(request).execute();
+        response.close();
+
+        assertEquals(301, response.code());
+        verifyNoInteractions(recorder);
+    }
+
+    @Test
+    void clientErrorResponse_recordsNetworkEvent() throws IOException {
+        server.enqueue(new MockResponse().setResponseCode(404));
+
+        Request request = new Request.Builder().url(server.url("/not-found")).build();
+        Response response = client.newCall(request).execute();
+        response.close();
+
+        assertEquals(404, response.code());
+        verify(recorder).recordNetworkEvent(
+                eq(Level.CRITICAL), eq("GET"), contains("/not-found"), eq("404"));
+        verify(recorder, never()).recordErrorEvent(any());
+    }
+
+    @Test
+    void serverErrorResponse_recordsNetworkEvent() throws IOException {
+        server.enqueue(new MockResponse().setResponseCode(500));
+
+        Request request = new Request.Builder().url(server.url("/error")).build();
+        Response response = client.newCall(request).execute();
+        response.close();
+
+        assertEquals(500, response.code());
+        verify(recorder).recordNetworkEvent(
+                eq(Level.CRITICAL), eq("GET"), contains("/error"), eq("500"));
+        verify(recorder, never()).recordErrorEvent(any());
+    }
+
+    @Test
+    void connectionFailure_recordsErrorEvent() {
+        server.enqueue(new MockResponse().setSocketPolicy(SocketPolicy.DISCONNECT_AT_START));
+
+        Request request = new Request.Builder().url(server.url("/fail")).build();
+
+        assertThrows(IOException.class, () -> client.newCall(request).execute());
+
+        verify(recorder).recordErrorEvent(any(IOException.class));
+        verify(recorder, never()).recordNetworkEvent(any(), any(), any(), any());
+    }
+
+    @Test
+    void postRequest_recordsCorrectMethod() throws IOException {
+        server.enqueue(new MockResponse().setResponseCode(500));
+
+        Request request = new Request.Builder()
+                .url(server.url("/post"))
+                .post(okhttp3.RequestBody.create("body", okhttp3.MediaType.parse("text/plain")))
+                .build();
+        Response response = client.newCall(request).execute();
+        response.close();
+
+        verify(recorder).recordNetworkEvent(eq(Level.CRITICAL), eq("POST"), any(), eq("500"));
+    }
+
+    @Test
+    void nullRecorder_errorResponse_doesNotThrowNPE() throws IOException {
+        server.enqueue(new MockResponse().setResponseCode(500));
+
+        OkHttpClient nullRecorderClient = new OkHttpClient.Builder()
+                .addInterceptor(new RollbarOkHttpInterceptor(null))
+                .build();
+
+        Request request = new Request.Builder().url(server.url("/error")).build();
+        Response response = nullRecorderClient.newCall(request).execute();
+        response.close();
+
+        assertEquals(500, response.code());
+    }
+
+    @Test
+    void nullRecorder_connectionFailure_doesNotThrow() {
+        server.enqueue(new MockResponse().setSocketPolicy(SocketPolicy.DISCONNECT_AT_START));
+
+        OkHttpClient nullRecorderClient = new OkHttpClient.Builder()
+                .addInterceptor(new RollbarOkHttpInterceptor(null))
+                .build();
+
+        Request request = new Request.Builder().url(server.url("/fail")).build();
+
+        assertThrows(IOException.class, () -> nullRecorderClient.newCall(request).execute());
+    }
+
+    @Test
+    void recorderThrowsOnErrorResponse_responseStillReturned() throws IOException {
+        server.enqueue(new MockResponse().setResponseCode(500));
+
+        doThrow(new RuntimeException("recorder boom"))
+                .when(recorder)
+                .recordNetworkEvent(any(), any(), any(), any());
+
+        Request request = new Request.Builder().url(server.url("/error")).build();
+        Response response = client.newCall(request).execute();
+        response.close();
+
+        assertEquals(500, response.code());
+    }
+
+    @Test
+    void recorderThrowsOnConnectionFailure_originalIOExceptionPropagates() {
+        server.enqueue(new MockResponse().setSocketPolicy(SocketPolicy.DISCONNECT_AT_START));
+
+        doThrow(new RuntimeException("recorder boom"))
+                .when(recorder)
+                .recordErrorEvent(any());
+
+        Request request = new Request.Builder().url(server.url("/fail")).build();
+
+        assertThrows(IOException.class, () -> client.newCall(request).execute());
+        verify(recorder).recordErrorEvent(any(IOException.class));
+    }
+}

settings.gradle.kts

@@ -38,6 +38,7 @@ include(
     ":rollbar-jakarta-web",
     ":rollbar-log4j2",
     ":rollbar-logback",
+    "rollbar-okhttp",
     ":rollbar-spring-webmvc",
     ":rollbar-spring6-webmvc",
     ":rollbar-spring-boot-webmvc",