search

dakrone / clj-http

An idiomatic clojure http client wrapping the apache client. Officially supported version.
http://clojars.org/clj-http
MIT License
1.78k stars 408 forks source link

Fixed decode-json-body #568

Closed ajchemist closed 3 years ago

ajchemist commented 3 years ago

Consider body charset

dakrone commented 3 years ago

Thanks @ajchemist, would you be able to add a test for this so it doesn't get accidentally removed in the future?

ajchemist commented 3 years ago

I'm not sure if it will be okay with https://github.com/dakrone/clj-http/pull/568/commits/a21d5b775f142ec412b210cc793df7d30c618b04.

rymndhng commented 3 years ago

The test looks good. Thanks!