return responseBody

HighKo · HighKo · commit 055a3c38953f · 2025-07-03T18:14:23.000+02:00
diff --git a/kotlin-insight-client/kotlin-insight-client-sdk/src/main/kotlin/com/linkedplanet/kotlininsightclient/sdk/SdkInsightAttachmentOperator.kt b/kotlin-insight-client/kotlin-insight-client-sdk/src/main/kotlin/com/linkedplanet/kotlininsightclient/sdk/SdkInsightAttachmentOperator.kt
@@ -72,10 +72,10 @@ object SdkInsightAttachmentOperator : InsightAttachmentOperator {
             val request = trustedGetRequestForCurrentUser(url).bind()
             val response = Either.catch { request.executeAndReturn<Response> { it } as Response }
                 .mapLeft { downloadFailed(500, url) }.bind()
-            if (response.isSuccessful) {
-                response.responseBodyAsStream
+            if (!response.isSuccessful) {
+                raise(downloadFailed(response.statusCode, url))
             }
-            raise(downloadFailed(response.statusCode, url))
+            response.responseBodyAsStream
         }
 
     private fun downloadFailed(statusCode: Int, url: String, ) = HttpInsightClientError(

Original file line number	Diff line number	Diff line change
`@@ -72,10 +72,10 @@ object SdkInsightAttachmentOperator : InsightAttachmentOperator {`
`72`	`72`	`val request = trustedGetRequestForCurrentUser(url).bind()`
`73`	`73`	`val response = Either.catch { request.executeAndReturn<Response> { it } as Response }`
`74`	`74`	`.mapLeft { downloadFailed(500, url) }.bind()`
`75`		`- if (response.isSuccessful) {`
`76`		`- response.responseBodyAsStream`
	`75`	`+ if (!response.isSuccessful) {`
	`76`	`+ raise(downloadFailed(response.statusCode, url))`
`77`	`77`	`}`
`78`		`- raise(downloadFailed(response.statusCode, url))`
	`78`	`+ response.responseBodyAsStream`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`private fun downloadFailed(statusCode: Int, url: String, ) = HttpInsightClientError(`