-
Notifications
You must be signed in to change notification settings - Fork 141
-
Notifications
You must be signed in to change notification settings - Fork 141
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
Do not know how to serialize a BigInt #1249
Comments
@brianpham93 looks like the issue is in JSON.stringify, after a quick internet search looks like some type replacement can be done to prevent this error, can you share some sample winston log we can use to reproduce? |
@brianpham93 Did you have any luck with your serialization issue? |
i'm using this working around temporarily BigInt.prototype.toJSON = function () {
return this.toString();
}; |
@brianpham93 Is there anything about your scenario that makes converting the Big Int to a string not work for you in the long term? |
@JacksonWeber So far i haven't found out any issue with current approach |
@brianpham93 Awesome! Mind if I mark this issue as complete in that case? |
@JacksonWeber No issue. It would be better if this is supported natively in app insights via a separate PR |
I use
winston
andwinston-azure-application-insights
to log directly to application insightsI'm getting this error when log an object that contains BigInt data
Does ApplicationInsights support BigInt or is there anything wrong with the way i'm logging data ?
Version: 2.9.1
Host: macOS 14, Apple Silicon
The text was updated successfully, but these errors were encountered: