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

Log improvement for faster debugging #49

Open
LasneF opened this issue Oct 6, 2023 · 1 comment
Open

Log improvement for faster debugging #49

LasneF opened this issue Oct 6, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@LasneF
Copy link

LasneF commented Oct 6, 2023

Given Wiretap as a proxy , Given that there is HTTP rewritting it would be nice to have better logging on the rewrite applied

here i got in wiretap logs this

level=info msg="[wiretap] request /trades: completed (307)" fileName=handle_request.go goroutine=686 package=daemon

would be nice to have received /trades , redirecting to http://ggg:zzz/agivenconfigurepath/toto/trades in the logs

could be also something to put in the UI either

here it s even not clear who reply the 307
looking on the ui , it propose a location https://localhost:8443/trades but looks not clear where does it comes from

@daveshanley
Copy link
Member

I think logging in general needs an upgrade, its very very noisy at the moment with no way to tune it. I will add this into the list for when I circle back for feature work in wiretap.

@daveshanley daveshanley added the enhancement New feature or request label Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants