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

Revisit gRPC's required usage of plaintext #42

Open
ljnelson opened this issue Oct 26, 2017 · 1 comment
Open

Revisit gRPC's required usage of plaintext #42

ljnelson opened this issue Oct 26, 2017 · 1 comment

Comments

@ljnelson
Copy link
Member

ljnelson commented Oct 26, 2017

In setting up the gRPC communication, I seem to recall there is a method call that sets up plaintext communication. Obviously this needs to be locked down; my recollection is that merely removing this method doesn't do the trick. Needs more research, particularly given that this project doesn't "own" the server (Tiller).

@ljnelson ljnelson added the bug label Oct 26, 2017
@ljnelson ljnelson added this to the Someday milestone Oct 26, 2017
@ljnelson ljnelson self-assigned this Oct 26, 2017
@ljnelson
Copy link
Member Author

Technically we're OK because you can always pass in your own ManagedChannel built however you like, but this would be a pretty common subclass operation I'd think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant