paulrutter commented on PR #522: URL: https://github.com/apache/felix-dev/pull/522#issuecomment-4969688563
Thanks Carsten, agreed on aligning the new ones. Since `org.apache.felix.http.require.config` already exists and can't change for backward compatibility, I'd align the new SNI properties to your `sni.required` form and group all three under a common `sni.` segment, keeping Jetty's own `sslContext`/`ssl` distinction for the TLS vs. HTTP level: - `org.apache.felix.https.sslContext.sni.required` (TLS level) - `org.apache.felix.https.ssl.sni.required` (HTTP level) - `org.apache.felix.https.ssl.sni.hostCheck` This keeps the three SNI properties consistent with each other (the host-check flag isn't a "require", so it fits the `sni.*` grouping better than a `require.*` form). Does that work for you? If so, I'll update the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
