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

[16.0] [ADD] fastapi_log #501

Draft
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

paradoxxxzero
Copy link
Contributor

This module allows an endpoint to activate full request logging in a database model.

It is useful to debug production issues or to monitor the usage of a specific endpoint.

This is a Work In Progress.

This is not a migration of rest_log, this module aims to be simpler, at least in the first iterations.

This PR depends on #500

@simahawk
Copy link
Contributor

Thanks for working on this. I wanted to do it since a while 😅

Early feedback...

The way I was planning to handle this was to have a base module to collect any kind of "request/response log".
This can be used by fastapi, base_rest, edi framework, endpoints and any custom implementation of request/response feature.

Would you consider splitting this part to a module like request_log or something like that?

@paradoxxxzero
Copy link
Contributor Author

@simahawk for now I don't want to overengineer this module. But feel free to split it when done. As a side note about that, is there any advantage to have all heterogeneous logs in the same model/views? Because aside of that, the amount of factored code would be low.

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