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

feat(node): Add AsyncLocalStorage implementation of AsyncContextStrategy #7800

Merged
merged 6 commits into from Apr 11, 2023

Conversation

timfish
Copy link
Collaborator

@timfish timfish commented Apr 9, 2023

This PR adds a new AsyncContextStrategy that uses AsyncLocalStorage.

This strategy is not enabled yet but there are tests to confirm it's functional.

This PR also include the change to ensureHubOnCarrier from #7799 which allow us to have correct scope inheritance.

@timfish timfish mentioned this pull request Apr 9, 2023
9 tasks
@timfish timfish added this to the Introduce AsyncContextManager milestone Apr 9, 2023
@timfish timfish self-assigned this Apr 9, 2023
@timfish timfish marked this pull request as ready for review April 11, 2023 07:08
@AbhiPrasad AbhiPrasad enabled auto-merge (squash) April 11, 2023 08:20
@AbhiPrasad AbhiPrasad merged commit 53ae9ae into getsentry:develop Apr 11, 2023
39 checks passed
@timfish timfish deleted the feat/async-hooks branch April 11, 2023 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants