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.

Sign up for GitHub

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

Jump to bottom

ArangoError throws when converted to JSON #632

Closed
neerajyadav opened this issue Oct 25, 2019 · 1 comment · Fixed by #626
Closed

ArangoError throws when converted to JSON #632

neerajyadav opened this issue Oct 25, 2019 · 1 comment · Fixed by #626
Labels
Feature Request Request for new functionality to be added to the driver. fixed Issue has been resolved but remains open due to a pending release. semver-minor This issue does not require any backwards-incompatible changes to address.
Milestone
v7

Comments

@neerajyadav
Copy link

If we try to JSON.stringify() the ArangoJsError it throws cyclic error.

Converting circular structure to JSON
    at JSON.stringify (<anonymous>)

This is either due to http-request or http-response objects included in the error thrown. One of them have self reference. Many of the logger frameworks uses JSON.stringify() method.
@pluma pluma changed the title cyclic error while JSON.stringify() the db server error ArangoError throws when converted to JSON Nov 18, 2019
@pluma pluma added the Feature Request Request for new functionality to be added to the driver. label Nov 18, 2019
@pluma pluma added the semver-minor This issue does not require any backwards-incompatible changes to address. label Apr 8, 2020
@pluma
Copy link
Contributor

pluma commented Apr 8, 2020

This issue will be addressed in the upcoming major release.

@pluma pluma linked a pull request Apr 8, 2020 that will close this issue
arangojs 7.0.0 #626
Merged
@pluma pluma added this to the v7 milestone Jun 18, 2020
@pluma pluma added the fixed Issue has been resolved but remains open due to a pending release. label Jul 6, 2020
@pluma pluma closed this as completed Aug 21, 2020
@pluma pluma reopened this Mar 1, 2021
@pluma pluma closed this as completed in 23336a8 Mar 12, 2021
@pluma pluma mentioned this issue Oct 20, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request Request for new functionality to be added to the driver. fixed Issue has been resolved but remains open due to a pending release. semver-minor This issue does not require any backwards-incompatible changes to address.
Projects
None yet
Milestone
v7
Development

Successfully merging a pull request may close this issue.

arangojs 7.0.0 arangodb/arangojs
2 participants
@pluma @neerajyadav