Skip to content

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

Merged
AbhiPrasad merged 6 commits intogetsentry:developfrom
timfish:feat/async-hooks
Apr 11, 2023
Merged

feat(node): Add AsyncLocalStorage implementation of AsyncContextStrategy#7800
AbhiPrasad merged 6 commits intogetsentry:developfrom
timfish:feat/async-hooks

Conversation

@timfish
Copy link
Copy Markdown
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
@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.

2 participants