-
Notifications
You must be signed in to change notification settings - Fork 96
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 logging- Feature request #278
Comments
Indeed. I’m curious to see what you have in mind. Would you like to work on that? It’ll need tests too. |
I was thinking of having a logging.conf, and then building off it in different modules. A separate folder for middleware, which automatically gets called before and after and after API call is made. It has some sort of unique ID for each request so you can trace which logs come from the same requests, and also time taken by API |
Not a bad idea. I have to see the first pass to make a comment on that. |
+1 on this is amazing! |
A basic logged added. |
If we add how logging can be incorporated, when API calls are made, using middleware it will cover all facets.
The text was updated successfully, but these errors were encountered: