fixup! improve Stats failure handling

binarynoise · binarynoise · commit d8f9fb63b1a0 · 2025-08-22T02:09:47.000+02:00
diff --git a/util/okhttp-kts/src/main/kotlin/okhttp-ktx.kt b/util/okhttp-kts/src/main/kotlin/okhttp-ktx.kt
@@ -199,7 +199,8 @@ fun OkHttpClient.postForm(
  */
 fun Response.checkSuccess() {
     if (code !in 200..399) {
-        throw HttpStatusCodeException(code, message)
+        readText(skipStatusCheck = true)
+        throw HttpStatusCodeException(code, message, this)
     }
     val location = getLocationUnchecked()
     if (location != null) {
@@ -211,7 +212,7 @@ fun Response.checkSuccess() {
     }
 }
 
-class HttpStatusCodeException(code: Int, message: String) : IllegalStateException("$code $message")
+class HttpStatusCodeException(val code: Int, message: String, val response: Response) : IllegalStateException("$code $message")
 
 /**
  * Retrieves the redirect location from the HTTP response.

Original file line number	Diff line number	Diff line change
`@@ -199,7 +199,8 @@ fun OkHttpClient.postForm(`
`199`	`199`	`*/`
`200`	`200`	`fun Response.checkSuccess() {`
`201`	`201`	`if (code !in 200..399) {`
`202`		`- throw HttpStatusCodeException(code, message)`
	`202`	`+ readText(skipStatusCheck = true)`
	`203`	`+ throw HttpStatusCodeException(code, message, this)`
`203`	`204`	`}`
`204`	`205`	`val location = getLocationUnchecked()`
`205`	`206`	`if (location != null) {`
`@@ -211,7 +212,7 @@ fun Response.checkSuccess() {`
`211`	`212`	`}`
`212`	`213`	`}`
`213`	`214`
`214`		`-class HttpStatusCodeException(code: Int, message: String) : IllegalStateException("$code $message")`
	`215`	`+class HttpStatusCodeException(val code: Int, message: String, val response: Response) : IllegalStateException("$code $message")`
`215`	`216`
`216`	`217`	`/**`
`217`	`218`	`* Retrieves the redirect location from the HTTP response.`