Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using clojure.data.json instead of cheshire #52

Open
rome-user opened this issue Jan 16, 2023 · 2 comments
Open

Using clojure.data.json instead of cheshire #52

rome-user opened this issue Jan 16, 2023 · 2 comments

Comments

@rome-user
Copy link

Hi.

I typically use clojure.data.json for JSON parsing, since cheshire is quite heavy and its transitive dependencies often get CVEs. Would you be interested in a patch that allows users to supply data.json as a JSON parsing library instead? As far as I can tell, this library does not use functionality specific to cheshire.

@fversnel
Copy link
Contributor

https://github.com/gnarroway/hato/blob/master/src/hato/middleware.clj#L69

json-encode and decode are dynamic so I guess you can override them in your own code.
Still it would be nice to also have built-in support for data.json

@rome-user
Copy link
Author

I'll write a patch soon. I have nothing against Cheshire, but I like being able to use data.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants