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
RPM: Add settings for HTTP/2 in sysconfig #115
Conversation
Warning: Untested, just a suggestion. |
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.
LGTM. CC @olamy since HTTP2 settings come from Winstone
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.
LGTM but http2 need more setup https://github.com/jenkinsci/winstone/#http2-support
@tn-osimis as I said in doc |
Add extra libraries directory setting and HTTP/2 documentation references
@olamy Oh not at all, I was indeed following this documentation. Just wanted to make sure everyone was fine with this being a Java 9+ thing only since I didn't include a setting for the So, I pulled the change in this PR accordingly. I'll try to build and test ASAP but if you preempt me I reckon this could be ready for merge. |
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.
LGTM
No description provided.