-
Notifications
You must be signed in to change notification settings - Fork 65
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
debug json parsing errors on valid json #620
Comments
jkieberk
pushed a commit
to jkieberk/yetibot
that referenced
this issue
Mar 25, 2017
-Idea from: mmazi/rescu#54 (comment) Not sure if this is something that clj-http should be stripping out on their end? This might just be a hack, and there is a better way to go about this? Also, adding [cheshire] as a dependency might fix it as well, although when I tried that, it still gave the same error (cheshire is a dependency of clj-http when using the :as :json feature of clj-http/get see [here](https://github.com/dakrone/clj-http#optional-dependencies) )
jkieberk
pushed a commit
to jkieberk/yetibot
that referenced
this issue
Mar 25, 2017
Woops, forgot to put this in a new branch
jkieberk
pushed a commit
to jkieberk/yetibot
that referenced
this issue
Mar 25, 2017
-Idea from: mmazi/rescu#54 Not sure if this is something that clj-http should be stripping out on their end? This might just be a hack, and there is a better way to go about this? Also, adding [cheshire] as a dependency might fix it as well, although when I tried that, it still gave the same error (cheshire is a dependency of clj-http when using the :as :json feature of clj-http/get see [here](https://github.com/dakrone/clj-http#optional-dependencies) )
jkieberk
pushed a commit
to jkieberk/yetibot
that referenced
this issue
Mar 25, 2017
-Idea from: mmazi/rescu#54 Not sure if this is something that clj-http should be stripping out on their end? This might just be a hack, and there is a better way to go about this? Also, adding [cheshire] as a dependency might fix it as well, although when I tried that, it still gave the same error (cheshire is a dependency of clj-http when using the :as :json feature of clj-http/get see [here](https://github.com/dakrone/clj-http#optional-dependencies) )
Fixed by #647 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
e.g.
The text was updated successfully, but these errors were encountered: