-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Turn off peer certificate verification for quic-server by default #7574
Turn off peer certificate verification for quic-server by default #7574
Conversation
0fa6434
to
7813c93
Compare
Signed-off-by: Ning Sun <[email protected]>
7813c93
to
65a47a3
Compare
@sunng87 the default configuration should be secure, so I think it should be left to true. What examples are not working for you? |
@sbordet by turning on this option, it requires any client to have a client certificate in secure connection handshake. This mechanism is typically used in mutual tls authentication. In most cases, like public https service, clients do not have such certificate and we do not auth client like this. So with current defaults, if we call the server with |
@sunng87 can you show how you initialize/configure your server? |
@joakime I'm using the code example from https://www.eclipse.org/jetty/documentation/jetty-10/programming-guide/index.html#pg-server-http-connector-protocol-http3 |
@sunng87 this is surprising. The server typically configures |
Yuck. Quiche requires this 😒 |
Merged to |
Current version of quic-server is configured to verify peer certificate (aka mutual tls) by default. I think this is accidentally set to
true
and should be changed tofalse
in order to get examples working.Signed-off-by: Ning Sun [email protected]