search

ring-clojure / ring-json

Ring middleware for handling JSON
313 stars 47 forks source link

big-decimals? interaction with other middleware. #32

Closed eraserhd closed 9 years ago

eraserhd commented 9 years ago

Looking at the code I noticed this: https://github.com/ring-clojure/ring-json/blob/master/src/ring/middleware/json.clj#L40

Shouldn't the binding of big-decimals be scoped to the parsing of the JSON, and not to chaining to other handlers?

weavejester commented 9 years ago

Yes, it probably should be. Patches welcome.