-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add logging to plugin #2
Conversation
This looks great! Thanks @dkreuer ! Only thing I might suggest, and this is very minor, is adding a new field to the log, maybe named Thanks again! |
You‘re right. Will add sth. like a context/phase whatever ;-) |
Updated logging with additional fields:
|
Super !!!!!!!!! |
This PR adds logging ability and log entries for request analyzing.
The configuration does not change since there already is a
LogLevel
config .. currently in use areDEBUG
- shows details of processingINFO
- shows outcome of analyzing and blockingERROR
- shows only errors in processingThis PR resolves #1 .
Example output in log (plugin logLevel debug, output contains access log from traefik too):