search

jooby-project / jooby

The modular web framework for Java and Kotlin
https://jooby.io
Apache License 2.0
1.7k stars 198 forks source link

RequestLogger with query params #805

Closed DrorBuhnik closed 7 years ago

DrorBuhnik commented 7 years ago

Currently RequestLogger log url path only without query params I look but didn't find if NCSA format define what exactly needs to go into the request part of the log

Can we add query params to the log? I can create pull request with the change (If I do, do you want it to be controllability, so the users can turn it on or off like they config the data format?)

jknack commented 7 years ago

Pull request with ability to turn on (off by default) will be great:

use(new RequestLogger().queryString());