Skip to content

PQC support backports for 3.105 - #7973

Merged
dralley merged 10 commits into
pulp:3.105from
dralley:3.105
Aug 14, 2026
Merged

PQC support backports for 3.105#7973
dralley merged 10 commits into
pulp:3.105from
dralley:3.105

Conversation

@dralley

@dralley dralley commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Generated-By: claude-opus-4.6
(cherry picked from commit 9a1bde6)
Use pysequoia instead, or shell out to gpg on the command line, which is
what python-gnupg does anyway. It's not much additional code to just
ditch the dependency everywhere.

Assisted-By: claude-opus-4.6
(cherry picked from commit bc6396a)
Restore python-gnupg to functional tests only - not worth trying to
replace this fixture like-for-like

Fix an issue with inlined signatures. We should only parse the sig
separately in the detached case.

(cherry picked from commit ac64ff3)
We want the lower bound to have PQC support.

(cherry picked from commit 2c01c5b)
The previous implementation counted fpr: lines in GPG's colon output to
verify that exactly one key matched the provided key ID. However, GPG
emits a separate fpr: line for the primary key and each subkey, so any
key with subkeys would be rejected with "There are N keys matching the
key id."

Count pub:/sec: lines instead, which represent actual distinct keys.

Assisted-By: claude-opus-4.6
(cherry picked from commit 9d26e95)
The add-signing-service management command previously hardcoded GPG for
key metadata extraction, preventing use with key types GPG cannot handle
(OpenPGP v6, ML-DSA/post-quantum). This adds a --backend option that
accepts "gpg" (default, existing behavior) or "sq" (Sequoia). Both
backends reuse --gnupghome and --keyring, mapped to the equivalent sq
CLI flags. The sq backend uses `sq cert export` to retrieve the public
key and pysequoia to parse the fingerprint.

Test infrastructure gains parallel Sequoia fixtures and helpers
(import_signing_key_sq, make_sq_signing_script, create_signing_service_sq)
and both signing service tests are parametrized to run with both backends.

Assisted-By: Claude Opus 4.6
closes pulp#7479

(cherry picked from commit a66a551)
Test scenarios added:

Certguard (parameterized over ML-DSA-65 and ML-DSA-87):
- Download with a valid PQC client cert returns 200
- Download with an untrusted PQC client cert returns 403
- Download with no client cert returns 403

Remote sync (ML-DSA-65):
- on_demand sync over HTTPS with PQC server certificate (TLS validation)
- on_demand sync over HTTPS with PQC mutual TLS (client cert required)

Static PQC cert/key assets and a generation script are included for both
ML-DSA-65 and ML-DSA-87. Certs are generated using pycryptography's x509
builder API.

Assisted-By: Claude Opus 4.6
(cherry picked from commit 03c3d76)
Post-quantum (ML-DSA) X.509 certificates are significantly larger than
traditional RSA/ECDSA certificates. When a reverse proxy forwards a
PQC client certificate via the X-CLIENT-CERT header, it can exceed
aiohttp's default 8190-byte max_field_size, causing a 400 LineTooLong
error. Increase the limit to 16KB.

Assisted-By: Claude Opus 4.6
(cherry picked from commit ab9a801)
@dralley

dralley commented Aug 14, 2026

Copy link
Copy Markdown
Contributor Author

Hmmm, the blocker here is that 3.105 isn't using the CentOS Stream 10 CI image, and therefore doesn't have the sq binary.

We normally would not swap the image of an older branch, but on the other hand we want full PQC testing, and that's really the only way to do it.

(cherry picked from commit 2d77388)
@dralley
dralley marked this pull request as ready for review August 14, 2026 05:08
@dralley
dralley merged commit 7d88f5f into pulp:3.105 Aug 14, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants