Skip to content
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 #5535 - Adding support for regex include/exclude of Protocols to SslContextFactory #5538

Merged
merged 4 commits into from Oct 30, 2020

Conversation

joakime
Copy link
Contributor

@joakime joakime commented Oct 30, 2020

Addresses both issue #5531 and #5535

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
…Factory

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
Jetty 9.4.34 automation moved this from Review in progress to Reviewer approved Oct 30, 2020
Copy link
Contributor

@sbordet sbordet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just remove the password and LGTM.

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@joakime joakime merged commit 1f727cd into jetty-9.4.x Oct 30, 2020
Jetty 9.4.34 automation moved this from Reviewer approved to Done Oct 30, 2020
@joakime joakime deleted the jetty-9.4.x-5531-ssl-dump-protocols branch October 30, 2020 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
2 participants