KEYCLOAK-17354 respond to tls CertificateRequest#7928
KEYCLOAK-17354 respond to tls CertificateRequest#7928sjweixel wants to merge 3 commits intokeycloak:mainfrom
Conversation
Set certificate to respond to CertificateRequest for ldaps and starttls requests when LDAP server requires client certificate verification.
|
Please review. |
|
@sjweixel Thanks for the PR! My vote is that we don't drive by the use of system properties, but instead we properly Keycloak SPI mechanism for this. Perhaps we can add 2 additional optional configuration options like "keystore-file" and "keystore-password" to the TrustStore provider and document it here https://www.keycloak.org/docs/latest/server_installation/index.html#_truststore ? Could you please send the mail to keycloak-dev mailing list for this topic? This will make sure that we reach bigger audience and have feedback from more people for this topic. |
|
@sjweixel Did you had a chance to look at this? |
|
My goal is to make setting the certificate for CertificateRequest easily configurable in keycloak-containers. I think it makes sense in to add these settings in truststore. I just need to be sure that keycloak-containers be able to be modified so CertificateRequest certificate will be as easy to setup in keycloak-containers as setting a tls certificate. I may not have time to look further into this for at least a month. |
Set certificate to respond to CertificateRequest for ldaps and starttls requests when LDAP server requires client certificate verification.