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

certificates: Extend the lifetime of non-web Engine certificates #329

Merged
merged 1 commit into from
May 6, 2022

Conversation

mz-pdm
Copy link
Member

@mz-pdm mz-pdm commented May 4, 2022

Non-web certificates are not subjects to the lifetime limitations
required by web browsers. Let’s extend the lifetime of non-web Engine
certificates to 5 years (the same value as for the host certificates)
to make the certificate updates much less frequent.

Let’s also drop the check for long certificate lifetime. All web
certificates should already have the shortened lifetime, otherwise
they wouldn’t work with current browsers.

Bug-Url: https://bugzilla.redhat.com/2079835

@mz-pdm
Copy link
Member Author

mz-pdm commented May 4, 2022

OVN certificates are not updated when running engine-setup in my environment so I don't whether that part works properly. But the change there is trivial.
I also don't know what ovirt-engine-rename is and how to test it.

@mwperina
Copy link
Member

mwperina commented May 6, 2022

I also don't know what ovirt-engine-rename is and how to test it.

ovirt-engine-rename is a tool to change FQDN of engine (for example after migration to a different physical host). In theory it should not only change FQDN wherever it's used, but also force to regenerate engine certificat. But this tools was never fully supported, so no need to perform some extensive testing

Non-web certificates are not subjects to the lifetime limitations
required by web browsers.  Let’s extend the lifetime of non-web Engine
certificates to 5 years (the same value as for the host certificates)
to make the certificate updates much less frequent.

Let’s also drop the check for long certificate lifetime.  All web
certificates should already have the shortened lifetime, otherwise
they wouldn’t work with current browsers.

Bug-Url: https://bugzilla.redhat.com/2079835
Copy link
Member

@michalskrivanek michalskrivanek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OST passed, certificates have different validity.

@michalskrivanek michalskrivanek merged commit 67e7763 into oVirt:master May 6, 2022
Copy link
Member

@didib didib left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re "Let’s also drop the check for long certificate lifetime. All web
certificates should already have the shortened lifetime, otherwise
they wouldn’t work with current browsers.":

What if they actually do not work with current browsers? We introduced this code only less than 1.5 years ago, in bz 1906320, 4.4.5.

People that:

  1. Set up 4.4.4
  2. Use old browsers
  3. Upgrade to 4.4.5
  4. Upgrade their browsers
    Will hopefully never notice any issue - everything will be seamless.

But people that do exactly the same, but in step (3.) upgrade to current version directly, will run into issues (popups/blocks/etc from their browsers), if I got it right. This is a regression. Am I missing something?

Also: If you didn't test ovirt-engine-rename, please ask QE to do at least some sanity testing of it. I know, @michalskrivanek, that I [1] try to present it as "not fully supported", but official docs [2] are less explicit...

[1] https://www.ovirt.org/develop/networking/changing-engine-hostname.html

[2] https://www.ovirt.org/documentation/administration_guide/index.html#sect-The_oVirt_Engine_Rename_Tool

@mz-pdm
Copy link
Member Author

mz-pdm commented May 9, 2022

What if they actually do not work with current browsers? We introduced this code only less than 1.5 years ago, in bz 1906320, 4.4.5.

People that:

  1. Set up 4.4.4
  2. Use old browsers
  3. Upgrade to 4.4.5
  4. Upgrade their browsers
    Will hopefully never notice any issue - everything will be seamless.

But people that do exactly the same, but in step (3.) upgrade to current version directly, will run into issues (popups/blocks/etc from their browsers), if I got it right. This is a regression. Am I missing something?

This was @michalskrivanek's suggestion so he should be best able to answer this.

@michalskrivanek
Copy link
Member

I think it's fine. 4.4.3 (that's where we changed it) is ~15 months old. A long time to upgrade browsers really.
And still, from what I've read it's applicable to public CAs and so our internal apache certificate shouldn't be refused

@mz-pdm mz-pdm deleted the certificates-engine branch May 24, 2022 11:49
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.

None yet

6 participants