fix(okhttp): Finalize OkHttpEvent even if no active span

Author: romtsn

sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpEvent.kt

@@ -27,7 +27,7 @@ internal class SentryOkHttpEvent(private val scopes: IScopes, private val reques
     internal val callSpan: ISpan?
     private var response: Response? = null
     private var clientErrorResponse: Response? = null
-    private val isEventFinished = AtomicBoolean(false)
+    internal val isEventFinished = AtomicBoolean(false)
     private var url: String
     private var method: String
 

sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpInterceptor.kt

@@ -192,6 +192,7 @@ public open class SentryOkHttpInterceptor(
 
     private fun finishSpan(span: ISpan?, request: Request, response: Response?, isFromEventListener: Boolean, okHttpEvent: SentryOkHttpEvent?) {
         if (span == null) {
+            okHttpEvent?.finish()
             return
         }
         if (beforeSpan != null) {

sentry-okhttp/src/test/java/io/sentry/okhttp/SentryOkHttpInterceptorTest.kt

@@ -650,4 +650,15 @@ class SentryOkHttpInterceptorTest {
         val okHttpEvent = SentryOkHttpEventListener.eventMap[call]!!
         assertEquals(fixture.server.url("/hello/v1").toUrl().toString(), okHttpEvent.callSpan!!.getData("url"))
     }
+
+    @Test
+    fun `when no active http span still finalizes okHttpEvent`() {
+        val client = fixture.getSut(isSpanActive = false, eventListener = SentryOkHttpEventListener(fixture.scopes))
+        val request = getRequest("/hello/")
+        val call = client.newCall(request)
+        call.execute()
+
+        val okHttpEvent = SentryOkHttpEventListener.eventMap[call]!!
+        assertTrue(okHttpEvent.isEventFinished.get())
+    }
 }