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.

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

Release v7.2.1 #722

Merged
merged 2 commits into from
Mar 29, 2021
Merged

Release v7.2.1 #722

merged 2 commits into from
Mar 29, 2021

Conversation

github-actions[bot]
Copy link
Contributor

Proposed Release Notes

  • Dev-only sub-dependency bump of 'y18n' to clear npm audit warnings.

  • Bumped @grpc/grpc-js to ^1.2.11.

  • Bumped @grpc/proto-loader to ^0.5.6.

  • Agent no longer propagates segments for promises via async-hooks when the transaction associated with the parentSegment has ended.

    This change reduces the amount of context tracking work needed for certain rare edge-case scenarios involving promises.

  • Fixed issue where capturing axios request errors could result in a memory leak.

    The agent now clears error references on transaction end, which are not used for later processing. Errors returned from 'axios' requests contain a reference to the request object which deeper down has a handle to a promise in handleRequestError. The TraceSegment associated with that promise has a handle to the transaction, which through the error capture ultimately kept the promise in memory and prevented it from being destroyed to free-up the TraceSegment from the segment map. This change also has the benefit of freeing up some memory early for transactions held onto for transaction traces.

  • Added active transaction check to wrappedResEnd to prevent unecessary work for ended transactions in the case of multiple Response.prototype.end() invocations.

Links

Details

@michaelgoin michaelgoin added risk: medium This change impacts senstive areas or has other traits that warrant more careful consideration. semver: patch Backwards compatible fixes. labels Mar 29, 2021
@michaelgoin michaelgoin marked this pull request as ready for review March 29, 2021 20:49
Copy link
Contributor

@carlo-808 carlo-808 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@michaelgoin michaelgoin reopened this Mar 29, 2021
@carlo-808 carlo-808 merged commit 5ae4a81 into main Mar 29, 2021
@carlo-808 carlo-808 deleted the release/v7.2.1 branch March 29, 2021 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
risk: medium This change impacts senstive areas or has other traits that warrant more careful consideration. semver: patch Backwards compatible fixes.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants