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/improve context #29

Merged
merged 2 commits into from
Nov 7, 2023
Merged

Feat/improve context #29

merged 2 commits into from
Nov 7, 2023

Conversation

Yehonal
Copy link
Member

@Yehonal Yehonal commented Nov 7, 2023

  • Integrated Nestjs-cls library for the global AsyncLocalStorage support
  • Transport headers from middleware into our API strategies
  • Inject appAlias into a static controller property to let interceptors and guards to access this information (useful for logging)
  • Created YalcASLService for module-related storage
  • Export dynamic getAppEventToken() and getAppLoggerToken() instead of the static APP_EVENT_TOKEN and APP_LOGGER_TOKEN, to fix the issues with the value overriding

@Yehonal Yehonal merged commit 50ba275 into dev Nov 7, 2023
3 checks passed
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.

1 participant