Merge pull request #2100 from Infomaniak/add-api-code-to-call-breadcrumb

Add api response code to api call sentry breadcrumb
Infomaniak · Nov 14, 2024 · 5534666 · 5534666
2 parents cb02c6d + 2a51c42
commit 5534666
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/app/src/main/java/com/infomaniak/mail/data/api/UrlTraceInterceptor.kt b/app/src/main/java/com/infomaniak/mail/data/api/UrlTraceInterceptor.kt
@@ -28,13 +28,21 @@ class UrlTraceInterceptor : Interceptor {
         var request = chain.request()
 
         val requestContextId = UUID.randomUUID().toString()
-        SentryDebug.addUrlBreadcrumb(request.url.toString(), requestContextId)
 
         request.newBuilder().apply {
             header("x-infomaniak-request-context-id", requestContextId)
             request = build()
         }
 
-        return chain.proceed(request)
+        val response = runCatching {
+            chain.proceed(request)
+        }.onFailure {
+            SentryDebug.addUrlBreadcrumb(request.url.toString(), requestContextId, null)
+            throw it
+        }.getOrThrow()
+
+        SentryDebug.addUrlBreadcrumb(request.url.toString(), requestContextId, response.code)
+
+        return response
     }
 }
diff --git a/app/src/main/java/com/infomaniak/mail/utils/SentryDebug.kt b/app/src/main/java/com/infomaniak/mail/utils/SentryDebug.kt
@@ -61,12 +61,13 @@ object SentryDebug {
         previousDestinationName = newDestinationName
     }
 
-    fun addUrlBreadcrumb(url: String, requestContextId: String) {
+    fun addUrlBreadcrumb(url: String, requestContextId: String, responseCode: Int?) {
         addInfoBreadcrumb(
             category = "API",
             data = mapOf(
                 "url" to url,
                 "requestContextId" to requestContextId,
+                "responseCode" to responseCode.toString(),
             ),
         )
     }