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

reqlog middleware #79

Open
mybigman opened this issue Aug 24, 2022 · 4 comments
Open

reqlog middleware #79

mybigman opened this issue Aug 24, 2022 · 4 comments
Assignees
@shraddhafalane
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@mybigman
Copy link

💡 The feature or bug you are proposing

[feature] reqlog middleware to be configurable

📄 The description of the bug or the logic behind your proposal

Are you able to make the logger configurable so it can include other parameters?

ip, path, referrer, agent, header etc etc
@vmihailenco vmihailenco added enhancement New feature or request good first issue Good for newcomers labels Aug 24, 2022
@shraddhafalane
Copy link

Would like to take this up

@vmihailenco
Copy link
Member

@shraddhafalane it is yours :)

@shraddhafalane shraddhafalane mentioned this issue Aug 25, 2022
Open
4 tasks
@shraddhafalane
Copy link

shraddhafalane commented Sep 1, 2022 •
edited
Loading

@vmihailenco could you please review the PR changes?

@mybigman
Copy link
Author

mybigman commented Sep 1, 2022

@shraddhafalane

You have made some good progress and its appreciated.

I believe we still need more options and also to be able to configure the output format.

Take a look at the logger in fibers middleware

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shraddhafalane shraddhafalane

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@vmihailenco @mybigman @shraddhafalane