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
Issue #6008 - fix RequestLog usage with filePath. #6033
Issue #6008 - fix RequestLog usage with filePath. #6033
Conversation
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The mod changes don't seem to be necessary.
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you were right to try to fix this. Leaving it as is is not great.
I think just removing the reference in the comment is OK. Using the ?= is mostly OK.
So pick one and I'll approve either this time.
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Closes #6008
Restore the previous usage of the
jetty.requestlog.filePath
property which was broken with #6022.