search

lumigo-io / lumigo-node

Lumigo's Node.js Distributed Tracing and Performance Agent
https://lumigo.io
Apache License 2.0
76 stars 12 forks source link

fix: handle BigInt serialization #512

Closed harelmo-lumigo closed 2 months ago

harelmo-lumigo commented 2 months ago

Fixes Issue

Addresses the issue from this thread, where the tracer fails when the event payload includes a BigInt value.

Changes proposed

Serialize the value as a numeric string

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.34%. Comparing base (4e438b5) to head (1fdd832). Report is 564 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #512 +/- ## =========================================== - Coverage 100.00% 99.34% -0.66% =========================================== Files 19 42 +23 Lines 1165 2734 +1569 Branches 246 761 +515 =========================================== + Hits 1165 2716 +1551 - Misses 0 17 +17 - Partials 0 1 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

doriaviram commented 2 months ago

:tada: This PR is included in version 1.100.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: