-
Notifications
You must be signed in to change notification settings - Fork 376
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
[Feature Request] Allow specifying TLS configuration #294
Comments
The capability is just an incremental change over existing chart, hence I have not submitted a formal proposal. |
Hi, I made a PR a long time ago but closed because they wanted to revamp the project ... (I'm still using it for myself and our organization because TLS is a prerequisite) |
@lerminou Coming accross the exact same issue at the moment and need to setup mTLS for the frontend SDK as a prerequisite - Will take a look at your PR 👍 |
What is the chances the original proposed PR can be re-opened? This proposal is now closed, which leads me to believe nobody internal is working on it. temporalio/proposals#42 |
I believe all aspects of this are now supported by the current chart. Please re-open if this is not the case and I will work with you to get whatever is missing sorted. |
Is your feature request related to a problem? Please describe.
The ability to specify and configure TLS options is documented (https://docs.temporal.io/docs/server/configuration/#tls) but not implemented in the charts
Describe the solution you'd like
Ability to configure and MTLS options in values.yaml
Additional context
The text was updated successfully, but these errors were encountered: