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 FTBFS on Ubuntu #194

Merged
merged 7 commits into from
Feb 20, 2025
Merged

Fix FTBFS on Ubuntu #194

merged 7 commits into from
Feb 20, 2025

Conversation

jelmer
Copy link
Member

@jelmer jelmer commented Feb 17, 2025

Imported from https://code.launchpad.net/~adrien/brz/brz/+merge/479089:

Breezy FTBFS in Ubuntu plucky due to tests, following the switch to python 3.13. This is a set of changes to solve that, plus a couple misc improvements.

The FTBFS was caused by two changes in python 3.13:

@jelmer jelmer changed the base branch from main to 3.3 February 17, 2025 11:50
@jelmer jelmer force-pushed the adrien branch 3 times, most recently from 32d63e3 to 73fef88 Compare February 19, 2025 12:12
adrien-n and others added 7 commits February 19, 2025 17:36
Python 3.13 has started requiring the authorityKeyIdentifier field as per
python/cpython#107361 . After iterating a bit,
it appears that we only need to pass "-addext keyUsage=keyCertSign" to
openssl during CA certificate creation and the server certificate will have
the proper field.

It's also possible to use something like `trustme` to generate the
certificates but that would have been a much larger change and more work
(maybe leading to most of the script being dropped however).

Forwarded: breezy-team#143
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1092458
While this is not a security concern since this is for a test CA, this is
future-proofing: DES3 is outdated and it is not impossible that it this
usage does not become possible anymore in the future.
@jelmer jelmer merged commit c35bc41 into breezy-team:3.3 Feb 20, 2025
6 checks passed
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