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.

Sign up for GitHub

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

Jump to bottom

Add function callback to pre/post invocation hook context #553

Closed
ejizba opened this issue Mar 14, 2022 · 0 comments · Fixed by #565
Closed

Add function callback to pre/post invocation hook context #553

ejizba opened this issue Mar 14, 2022 · 0 comments · Fixed by #565
Assignees
@ejizba
Labels
enhancement P1
Milestone
April 2022

Comments

@ejizba
Copy link
Contributor

ejizba commented Mar 14, 2022

Met with App Insights folks and the main thing missing from the new hooks is the function callback. The callback is an argument to cls-hooked in their wrapWithCorrelationContext method.

Since we store function callbacks across executions, we should make sure the hooks for one execution can't affect another execution.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ejizba ejizba

Labels
enhancement P1
Projects
None yet
Milestone
April 2022
Development

Successfully merging a pull request may close this issue.

Add functionCallback to preInvocationContext Azure/azure-functions-nodejs-worker
1 participant
@ejizba