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

[Feature] Add the possibility to add CA certificates as volumes/config #127

Closed
2 tasks
fforootd opened this issue Sep 5, 2023 · 7 comments · Fixed by #205
Closed
2 tasks

[Feature] Add the possibility to add CA certificates as volumes/config #127

fforootd opened this issue Sep 5, 2023 · 7 comments · Fixed by #205
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@fforootd
Copy link
Member

fforootd commented Sep 5, 2023

In some scenarios customers want to configure zitadel to send outbound traffic through a proxy server.

We already support defining HTTP_PROXY settings but there is no way mounting a CA file

Acceptance Criteria

  • Users can mount their CA file (this should mount /etc/ssl/certs/)
  • HTTPS_PROXY can use the CA file
@fforootd fforootd added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Sep 5, 2023
@unique-dominik
Copy link

unique-dominik commented Feb 27, 2024

@fforootd Is it enough to just mount the directory/file if we do not use the proxy? Can we also set the SSL_CERT_FILE environment variable to tell Zitadel to use it. Same here btw, if I make it work at the bank, a meaty PR awaits 😂

@unique-dominik
Copy link

Self-reply 1 for the docs: Yes, it is enough to just mount them like:

extraVolumeMounts:
  - name: ca-certs
    mountPath: /etc/ssl/certs/ca-certs.crt
    readOnly: true
    
extraVolumes:
  - name: ca-certs
    secret:
      defaultMode: 420
      secretName: ca-certs

@fforootd
Copy link
Member Author

Nice, thanks for sharing, I think this becomes more relevant with each day 😁

@unique-dominik
Copy link

We are deploying at an FSI customer on premise and we need to fumble the chart a bit (user can never be root, fs changes must be made via securityContext etc. Once I have that stuff running I try to PR it if you are not faster.

@eliobischof
Copy link
Member

eliobischof commented May 8, 2024

This feature was just released

No, the feature for an inbound cert was released

@unique-dominik
Copy link

ℹ️ Not forgotten still, I track it in our JIRA to bring the code back here 🕐

@fforootd
Copy link
Member Author

fforootd commented May 8, 2024

ℹ️ Not forgotten still, I track it in our JIRA to bring the code back here 🕐

Thanks for letting us know!

cr1cr1 added a commit to thedataflows/zitadel-charts that referenced this issue May 23, 2024
eliobischof pushed a commit that referenced this issue May 24, 2024
Add extraVolumes and extraVolumeMounts; closes #127
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants