Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants