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

Noisy logging #1508

Open
ptman opened this issue Sep 4, 2020 · 2 comments
Open

Noisy logging #1508

ptman opened this issue Sep 4, 2020 · 2 comments
Labels
effort-minutes Will take up to 60 minutes to implement priority-should sooner rather than later
Milestone
Backlog

Comments

@ptman
Copy link
Contributor

ptman commented Sep 4, 2020

revel/router.go

Line 754 in ee237a4

log.Infof("Reversing action %s to %s Using Route %#v",action,urlPath,pathData.Route)

Could the logging be removed or changed to debug?
@brendensoares brendensoares added effort-minutes Will take up to 60 minutes to implement priority-should sooner rather than later labels Apr 13, 2022
@brendensoares brendensoares added this to the Backlog milestone Apr 13, 2022
@brendensoares
Copy link
Member

That seems like a reasonable suggestion. Can you create a PR?

@ptman
Copy link
Contributor Author

ptman commented Apr 13, 2022

This was already dropped from Info to Debug. Should we remove it completely?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
effort-minutes Will take up to 60 minutes to implement priority-should sooner rather than later
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
2 participants
@ptman @brendensoares