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 an example of using Dropwizard with Conscrypt #2230

Merged
merged 1 commit into from Dec 14, 2017

Conversation

Projects
None yet
4 participants
@arteam
Member

arteam commented Dec 13, 2017

Problem:

Jetty 9.4.8 supports native SSL via Conscrypt, but it's not clear how to use it in Dropwizard.

Solution:

Add an integration test and upgrade the SSL documentation.

Result:

Dropwizard users can see how they can take advantage of native SSL.

@arteam arteam added the improvement label Dec 13, 2017

@arteam arteam added this to the 1.3.0 milestone Dec 13, 2017

Add an example of using Dropwizard with Conscrypt
Conscrypt is used as the JCE provider for a HTTP/2 app.

@arteam arteam force-pushed the conscrypt branch from 8b78632 to 6050574 Dec 13, 2017

@jplock

jplock approved these changes Dec 14, 2017

@jplock jplock merged commit 9106a3b into master Dec 14, 2017

5 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@jplock jplock deleted the conscrypt branch Dec 14, 2017

@GautamGupta

This comment has been minimized.

Contributor

GautamGupta commented Mar 10, 2018

It's probably also a good idea to link to that part of the documentation in the configuration section (or to update it).

@joschi

This comment has been minimized.

Member

joschi commented Mar 13, 2018

@GautamGupta Good idea! Would you mind creating a pull request for it?

@GautamGupta

This comment has been minimized.

Contributor

GautamGupta commented Mar 13, 2018

Sure thing, here you go: #2305

joschi added a commit that referenced this pull request Mar 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment