-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Http2 support #514
base: master
Are you sure you want to change the base?
Http2 support #514
Conversation
…ht version of alnp library to boot classpath.
@gimlet2 Do you know in which version of |
This is great. @gimlet2, could you update this? |
@tipsy, updated |
*/ | ||
public class HelloSecureHttp2World { | ||
public static void main(String[] args) { | ||
secure(args[0], args[1], null, null); |
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.
secure(SparkTestUtil.getKeyStoreLocation(), SparkTestUtil.getKeystorePassword(), null, null);
How can I do this? Can you add some tests? |
@tipsy Are we just missing tests? Is that the only thing blocking this from delivery? |
@tdalbo92 From what I can see, yes. But @perwendel needs to merge this one. |
@tipsy beautiful, thanks for the update. I'm interested in getting this in as part of a new project I'm writing. If there's help needed with the tests, I can give it my best to write some. |
Any news on this? I'd love to be able to use HTTP2 for a site I'm working on. |
any news on that? Would love to use http2 instead of websockets ! |
I'm baffled by this. |
Any updates? |
Any progress on getting in http2 support done. |
I did a new PR from where gimlet2 left, added the tests and updated the code so it works with the current Jetty version and also added support for HTTP 2 clear protocol #1183 |
Http2 support added. To be able to use http2 pay attention to add right version of alnp library to boot classpath.