-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add trace middleware #216
Add trace middleware #216
Conversation
Also set it as the outer most middleware since logIDs are needed by inner middlewares
Codecov ReportBase: 72.76% // Head: 72.76% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #216 +/- ##
=======================================
Coverage 72.76% 72.76%
=======================================
Files 34 34
Lines 2214 2214
=======================================
Hits 1611 1611
Misses 523 523
Partials 80 80
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
This middleware is now the first middleware in the chain. This is necessary because there are other inner middlewares that depend on logIDs.