-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Ensure CA provider config changes result in certificate regeneration across all CA providers #9572
Comments
… CA will regenerate the roots progress on #9572
…/bits for the vault connect CA provider progress on #9572
… CA will regenerate the roots progress on #9572
…/bits for the vault connect CA provider progress on #9572
… CA will regenerate the roots progress on #9572
…/bits for the vault connect CA provider progress on #9572
Hi @preetapan , should the changes in private-key-bits always result in the root CA (as opposed to the consul controlled intermediate CA) requiring a reconfiguration? My example is: I have a Root CA that has EC, 521 bit keys for its certificate while my Intermediate still EC, used 256 bit keys and generates leaf certs with the same EC 256. With the changes in #10331 I now get the error of |
Thank you for reporting this problem! I just wrote up #12246 which I believe may describe the problem you encountered. Given these problems, I think we may need to either change this validation, or remove it. |
Currently, changes to two different fields
private-key-bits
andprivate-key-type
in the CA provider config for Consul connect, don't result in regenerating root certificates.This results in confusing UX because changes don't take effect after the config is updated , and leads to mismatched expectations particularly when the changes affect the type of encryption used etc.
All providers implement this logic differently, and we will need to create follow up issues per provider to make sure nuances like the intermediate mount point in the Vault CA provider is updated correctly.
The text was updated successfully, but these errors were encountered: