Skip to content

Commit

Permalink
Fix: Catch JSON Parse Errors in APIUtils (#2819)
Browse files Browse the repository at this point in the history
  • Loading branch information
ptlthg authored Oct 26, 2024
1 parent 1f6b099 commit f226fc1
Showing 1 changed file with 10 additions and 3 deletions.
13 changes: 10 additions & 3 deletions src/main/java/at/hannibal2/skyhanni/utils/APIUtils.kt
Original file line number Diff line number Diff line change
Expand Up @@ -181,9 +181,16 @@ object APIUtils {

private fun readResponse(entity: HttpEntity): JsonObject {
val retSrc = EntityUtils.toString(entity) ?: return JsonObject()
val parsed = parser.parse(retSrc)
if (parsed.isJsonNull) return JsonObject()
return parsed as JsonObject

try {
val parsed = parser.parse(retSrc)
if (parsed.isJsonNull) return JsonObject()

return parsed as JsonObject
} catch (_: Throwable) {
// This causes content types that aren't JSON to be ignored
return JsonObject()
}
}

fun postJSONIsSuccessful(url: String, body: String, silentError: Boolean = false): Boolean {
Expand Down

0 comments on commit f226fc1

Please sign in to comment.