-
Notifications
You must be signed in to change notification settings - Fork 3.9k
More flexible SSL configuration API #1881
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
Comments
We didn't originally because I think tcnative required a File (or we thought it required a File) and we didn't want to save the private key InputStream to a temporary file without the user's knowledge. This may be possible now. |
Looks like it is indeed possible now: |
Also update unit tests to stop writing certs to temp files. Fixes grpc#1881
@zpencer Could I ask why your PR was closed? Is something in the works on this issue? |
I think with #4316 this was completed. I will note though that we also have the newer Credentials API that also allows using InputStreams but also configuring many more TLS parameters. |
Today
ServerBuilder.useTransportSecurity
takes two files as parameter. However, sometimes onlyInputStream
s for the key and cert files are available. Can we add an API to take two input streams?The text was updated successfully, but these errors were encountered: