-
Notifications
You must be signed in to change notification settings - Fork 65
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
Implement 'query-log' #410
base: master
Are you sure you want to change the base?
Conversation
Could we add something so that the EDNS-ECS address is display in the log as well, or instead of the client-ip? |
Can certainly be done. To facilitate more values getting added, it's probably best to also switch to structured logging rather than using more columns. |
Added that, the output looks like this now:
Wondering if it'd be better to output these as JSON, or at least offer that as an option |
Having plain-text and json as (both) options would be great. Tailing a json file is a bit tricky if you want to monitor (where I use it for). |
Added support for JSON output (optional). This produces JSONL which can be piped into |
I think this is fine to merge. No issues at all last months when using it all the time. |
Implements #409
Adds a new
query-log
element that can log to file or STDOUT.