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

TLS configuration API (phase 1) #2285

Closed
lei-tang opened this issue Mar 16, 2022 · 7 comments
Closed

TLS configuration API (phase 1) #2285

lei-tang opened this issue Mar 16, 2022 · 7 comments

Comments

@lei-tang
Copy link
Contributor

lei-tang commented Mar 16, 2022

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)

Describe the feature request
The feature is described in its design document.

Describe alternatives you've considered
Alternatives are discussed in the design document.

Affected product area (please put an X in all that apply)

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X ] Security
[ ] Test and Release
[ ] User Experience

Additional context

@SpecialYang
Copy link
Member

How can I set global tls version for all gateways? By providing default tls version for gateways, users can also explicitly set tls version in specified gateway resource with high priority.
I am little confused that why this api can not applied on ingress and egress gateways for consistent user experience.

@lei-tang
Copy link
Contributor Author

lei-tang commented May 16, 2022

TLS version for gateways can be configured through minProtocolVersion and maxProtocolVersion of ServerTLSSettings( https://istio.io/latest/docs/reference/config/networking/gateway/#ServerTLSSettings). This API is different from the TLS API on gateways: this API is for TLS between workloads in a mesh whereas the the TLS API on gateways is for the external traffic to gateway servers (hence the name ServerTLSSettings).

@SpecialYang
Copy link
Member

SpecialYang commented May 17, 2022

Sometimes, we may hope to restrict tls version for all ingress gateways via meshconfig instead of doing duplicated works on every gateway resource.

@lei-tang
Copy link
Contributor Author

I concur that there is value to restrict TLS version for all ingress gateways through a single configuration.

@costinm
Copy link
Contributor

costinm commented May 17, 2022

I wouldn't mind a mechanism to provide defaults - but MeshConfig is not the right place. Keeping in mind we plan to move to the K8S Gateway API, which defines a much cleaner policy attachment mechanism: this would map to a GatewayClass attached policy.

Ideally such policy would be discussed in the K8S WG, so other implementations of the API use the same policy. We expect Istio users to integrate and use other gateway implementations ( for example in cases of global load balancing, etc ) - so policies that are cross-vendor are best.

IMO another hack in MeshConfig would be a move in the wrong direction and create yet another migration problem.

@SpecialYang
Copy link
Member

Ideally such policy would be discussed in the K8S WG, so other implementations of the API use the same policy. We expect Istio users to integrate and use other gateway implementations ( for example in cases of global load balancing, etc ) - so policies that are cross-vendor are best.

Make sense. Now, do we have some relative documents that demonstrate how to integrate other gateway implementations with istio.

@howardjohn
Copy link
Member

Work here is done

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

5 participants