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

Support key and trust store type in SimpleSSLContextFactory #938

Closed
sleberknight opened this issue Apr 14, 2023 · 0 comments · Fixed by #968
Closed

Support key and trust store type in SimpleSSLContextFactory #938

sleberknight opened this issue Apr 14, 2023 · 0 comments · Fixed by #968
Assignees
Labels
enhancement A request for change or improvement to an existing feature
Milestone

Comments

@sleberknight
Copy link
Member

  • Add keyStoreType
  • Add trustStoreType
  • The key and trust store type should default to "JKS" if not specified explicitly
  • Add support to create new instances with key and trust store type via an all-args constructor, or via the builder; existing constructors should be updated to set the default key/trust store type to "JKS"

This originated from discussion #935 (Enhancements to SSL/TLS configuration and context classes)

@sleberknight sleberknight added the enhancement A request for change or improvement to an existing feature label Apr 14, 2023
sleberknight added a commit that referenced this issue Apr 26, 2023
* Support trustStoreType in SSLContextConfiguration (#936)
* Support trustStoreType in SecureEndpointsConfiguration (#937)
* Support key and trust store type in SimpleSSLContextFactory (#938)
* Support trustStoreType in TlsContextConfiguration (#939)

Closes #936
Closes #937
Closes #938
Closes #939
@sleberknight sleberknight self-assigned this Apr 26, 2023
@sleberknight sleberknight added this to the 2.6.0 milestone Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A request for change or improvement to an existing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant