-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
unsupported value: +Inf" error not handled gracefully #20249
Comments
davidby-influx
added a commit
that referenced
this issue
Dec 4, 2020
JSON marshalling errors should be returned properly formatted in JSON like other errors. This fix formats marshalling errors the same way influxdb formats other query errors. Fixes #20249
This was referenced Dec 4, 2020
davidby-influx
added a commit
that referenced
this issue
Dec 4, 2020
JSON marshalling errors should be returned properly formatted in JSON like other errors. This fix formats marshalling errors the same way influxdb formats other query errors. Fixes #20249
davidby-influx
added a commit
that referenced
this issue
Dec 4, 2020
JSON marshalling errors should be returned properly formatted in JSON like other errors. This fix formats marshalling errors the same way influxdb formats other query errors. Fixes #20249
Pushed. ttps://github.com//pull/20250 |
4 tasks
davidby-influx
added a commit
that referenced
this issue
Dec 7, 2020
…0276) * fix(error): unsupported value: +Inf" error not handled gracefully JSON marshalling errors should be returned properly formatted in JSON like other errors. This fix formats marshalling errors the same way influxdb formats other query errors. Fixes #20249 (cherry picked from commit 2407077) * chore(changelog): update CHANGELOG.md for PR #20276
dgnorton
pushed a commit
that referenced
this issue
Feb 3, 2021
…0276) * fix(error): unsupported value: +Inf" error not handled gracefully JSON marshalling errors should be returned properly formatted in JSON like other errors. This fix formats marshalling errors the same way influxdb formats other query errors. Fixes #20249 (cherry picked from commit 2407077) * chore(changelog): update CHANGELOG.md for PR #20276
dgnorton
pushed a commit
that referenced
this issue
Feb 3, 2021
…0276) * fix(error): unsupported value: +Inf" error not handled gracefully JSON marshalling errors should be returned properly formatted in JSON like other errors. This fix formats marshalling errors the same way influxdb formats other query errors. Fixes #20249 (cherry picked from commit 2407077) * chore(changelog): update CHANGELOG.md for PR #20276
lesam
added a commit
to lesam/influxdb
that referenced
this issue
Mar 4, 2021
If there is some error writing to the response writer, we would previous have infinite recursion. Re-closes influxdata#20249
lesam
added a commit
to lesam/influxdb
that referenced
this issue
Mar 4, 2021
If there is some error writing to the response writer, we would previous have infinite recursion. Re-closes influxdata#20249
lesam
added a commit
to lesam/influxdb
that referenced
this issue
Mar 4, 2021
If there is some error writing to the response writer, we would previous have infinite recursion. Re-closes influxdata#20249
lesam
added a commit
to lesam/influxdb
that referenced
this issue
Mar 4, 2021
If there is some error writing to the response writer, we would previous have infinite recursion. Re-closes influxdata#20249
davidby-influx
pushed a commit
that referenced
this issue
Mar 6, 2021
* Revert "fix(error): unsupported value: +Inf" error not handled gracefully (#20250)" This reverts commit 6ac0bb3. * fix: No infinite recursion on write error If there is some error writing to the response writer, we would previous have infinite recursion. Re-closes #20249 (cherry picked from commit d6f7716)
davidby-influx
pushed a commit
that referenced
this issue
Mar 10, 2021
* Revert "fix(error): unsupported value: +Inf" error not handled gracefully (#20250)" This reverts commit 6ac0bb3. * fix: No infinite recursion on write error If there is some error writing to the response writer, we would previous have infinite recursion. Re-closes #20249 (cherry picked from commit d6f7716)
4 tasks
davidby-influx
added a commit
that referenced
this issue
Mar 10, 2021
* fix: infinite recursion bug (#20862) * Revert "fix(error): unsupported value: +Inf" error not handled gracefully (#20250)" This reverts commit 6ac0bb3. * fix: No infinite recursion on write error If there is some error writing to the response writer, we would previous have infinite recursion. Re-closes #20249 (cherry picked from commit d6f7716) * chore: update CHANGELOG.md Co-authored-by: Sam Arnold <sarnold@influxdata.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
JSON marshalling errors should be returned as properly formatted JSON errors.
Steps to reproduce:
Expected behavior:
Actual behavior:
Environment info:
The text was updated successfully, but these errors were encountered: