Skip to content

Expose set_groups functions#1477

Open
schwabe wants to merge 1 commit intopyca:mainfrom
schwabe:schwabe/set_groups
Open

Expose set_groups functions#1477
schwabe wants to merge 1 commit intopyca:mainfrom
schwabe:schwabe/set_groups

Conversation

@schwabe
Copy link
Contributor

@schwabe schwabe commented Feb 11, 2026

This allows pyOpenSSL to restrict the groups allowed to be used. E.g. for restricting the groups to post-quantum hybrid groups (e.g. X25519MLKEM768) to always ensure that post-quantum cryptography is used.

This commit uses the set_groups as public API since that is the preferred naming. Internally for the API we use the set_curves API since this name is available on all OpenSSL implementations and OpenSSL forks.

@schwabe
Copy link
Contributor Author

schwabe commented Feb 11, 2026

CI run with a patch that replaces dependency for cryptography with a hard-codeded reference to the cryptgraphy branch: https://github.com/schwabe/pyopenssl/actions/runs/21896043135/job/63212302434

@schwabe
Copy link
Contributor Author

schwabe commented Feb 13, 2026

Since the PR in cryptography was accepted and the version that this will end up in will be 47.0.0, I changed the requirements in setup.py to reflect this but the CI is not very happy about that (probably until 47.0.0 is actually released).

@alex
Copy link
Member

alex commented Feb 13, 2026

Yeah, that's exactly right.

@schwabe schwabe force-pushed the schwabe/set_groups branch from 6d55393 to 7dcdd50 Compare March 17, 2026 16:15
This allows pyOpenSSL to restrict the groups allowed to be used. E.g.
for restricting the groups to post-quantum hybrid groups
(e.g. X25519MLKEM768) to always ensure that post-quantum cryptography is
used.

This commit uses the set_groups as public API since that is the
preferred naming. Internally for the API we use the set_curves API
since this name is available on all OpenSSL implementations and
OpenSSL forks.

Signed-off-by: Arne Schwabe <arne@rfc2549.org>
@schwabe schwabe force-pushed the schwabe/set_groups branch from 7dcdd50 to 44ca856 Compare March 17, 2026 16:22
@schwabe
Copy link
Contributor Author

schwabe commented Mar 17, 2026

I updated the PR after the pyOpenSSL release to have the Changelog.rst again in a proper state and not add to the already released version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants