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

fix: use ca_cert instead of ca_cert_file #1164

Merged
merged 1 commit into from
Dec 30, 2024
Merged

Conversation

zucchinidev
Copy link
Contributor

The Credhub ca_cert_file was not being generated during installation. Instead, the Ops Manager certificate was being included in an unexpected configuration key.

Previously, the broker app ran on Diego and utilized the Cloud Foundry certificate trust store (the Ops Manager CA certificate was located in the container’s /etc/cf-system-certificates path).

Now, the broker runs on its own VM without automatic injection of the Ops Manager root CA or CF trust store. Although ca_cert was present in the job configuration, the broker wasn’t utilizing it, leading to the following error: tls: failed to verify certificate: x509: certificate signed by unknown authority.

Checklist:

  • Have you added or updated tests to validate the changed functionality?
  • Have you added Release Notes in the docs repositories?
  • Have you followed the Conventional Commits specification?

The Credhub ca_cert_file was not being generated during installation. Instead, the Ops Manager certificate was being included in an unexpected configuration key.

Previously, the broker app ran on Diego and utilized the Cloud Foundry certificate trust store (the Ops Manager CA certificate was located in the container’s /etc/cf-system-certificates path).

Now, the broker runs on its own VM without automatic injection of the Ops Manager root CA or CF trust store. Although ca_cert was present in the job configuration, the broker wasn’t utilizing it, leading to the following error:
`tls: failed to verify certificate: x509: certificate signed by unknown authority.`
@zucchinidev zucchinidev force-pushed the known_issue_credhub_cert branch from ae31754 to a7546cd Compare December 27, 2024 16:47
@zucchinidev zucchinidev merged commit fed7e56 into main Dec 30, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

1 participant