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 register a key using secret creationTimestamp instead of certificate validity timestamp #1681

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sybernatus
Copy link

Description of the change

Register a key and order it based on its secret creation timestamp instead of its cert starting validity (NotBefore attribute)

Benefits

In case we are bringing our own certificate, if the certificate is created before sealed secret installation but applied after the installation, still the latest certificate used will be the certificate installed by sealed secret during its initialization instead of the latest secret certificate created.

Possible drawbacks

Applicable issues

Additional information

…ate validity timestamp

Signed-off-by: Jérôme GARCIA <[email protected]>
@alvneiayu
Copy link
Collaborator

hi @sybernatus

First of all, thanks a lot for your PR.

This PR will change the way to order the certificates for all the people with Sealed Secrets deployed. This impact is really high if we include it as default.

I suggest to include it with a flag to configure what is the way that the users want to order the certificates. If we see in the future that everybody is using the flag, we can deprecated it and set it by default but I think that we need to be conservative here.
I though something like --order-by-creation-timestamp.

For this case, we need to include a test and documentation.

Thanks a lot

Álvaro

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants