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

renewal: fix key_type not being preserved on <v1.25.0 renewal configs #9636

Merged
merged 1 commit into from
Mar 28, 2023

Conversation

alexzorin
Copy link
Collaborator

Fixes #9635.

We may wish to backport this to Certbot 2.1.0 in order to have it land in https://packages.debian.org/bookworm/python3-certbot.

Copy link
Member

@bmw bmw left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks Alex.

@bmw bmw merged commit e10e549 into master Mar 28, 2023
@bmw bmw deleted the fix-renew-key-preservation branch March 28, 2023 15:44
@bmw bmw added this to the 2.5.0 milestone Mar 28, 2023
@bmw
Copy link
Member

bmw commented Mar 28, 2023

@hlieberman, I think you should try to backport this to the Certbot package in bookworm. Is that feasible?

Without this, when people update to Certbot in bookworm from an older version of Debian and the included cron job/systemd timer runs and renews their certificates, their certificates will switch from using RSA keys to ECDSA keys which definitely wasn't our intention. This PR seems to apply cleanly to Certbot 2.1.0 after doing something about changelog conflicts and also applying this minor change.

@hlieberman
Copy link
Member

@hlieberman, I think you should try to backport this to the Certbot package in bookworm. Is that feasible?

Definitely. Just sent 2.1.0-3 with that patch in it; will take about 10 days or so to make its way into bookworm.

@bmw
Copy link
Member

bmw commented Mar 29, 2023

Thanks so much for the quick response and help Harlan. I think you just prevented a lot of headaches.

@alexzorin
Copy link
Collaborator Author

@hlieberman I am seeing the patch in Debian sid but not yet bookworm, is everything still tracking OK for its inclusion?

One of our users encountered this bug today, unfortunately.

@hlieberman
Copy link
Member

hlieberman commented Apr 25, 2023 via email

@alexzorin
Copy link
Collaborator Author

Thanks for the update! I think it will be fine to wait the remaining 10 days.

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.

Certbot 2.x renew incorrectly changes key type to ECDSA if key_type is absent from renewal conf
3 participants