aws / aws-xray-sdk-node

The official AWS X-Ray SDK for Node.js.
Apache License 2.0
267 stars 156 forks source link

fix #594 memory leaks caused by cls-hooked #595

Closed regevbr closed 12 months ago

regevbr commented 1 year ago

*Issue #594

Description of changes:

Add option to set manual mode from env variable AWS_XRAY_MANUAL_MODE

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov-commenter commented 12 months ago

Codecov Report

Patch coverage has no change and project coverage change: +0.02% :tada:

Comparison is base (2a9eaa8) 83.38% compared to head (16965b5) 83.41%. Report is 9 commits behind head on master.

:exclamation: Current head 16965b5 differs from pull request most recent head ca66b78. Consider uploading reports for the commit ca66b78 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #595 +/- ## ========================================== + Coverage 83.38% 83.41% +0.02% ========================================== Files 37 37 Lines 1794 1797 +3 ========================================== + Hits 1496 1499 +3 Misses 298 298 ``` [see 1 file with indirect coverage changes](https://app.codecov.io/gh/aws/aws-xray-sdk-node/pull/595/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws)

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