Skip to content

TLS::Text_Policy::allowed_signature_schemes() - #5773

Merged
reneme merged 3 commits into
masterfrom
rene/tls_text_policy_for_signature_schemes
Jul 29, 2026
Merged

TLS::Text_Policy::allowed_signature_schemes()#5773
reneme merged 3 commits into
masterfrom
rene/tls_text_policy_for_signature_schemes

Conversation

@reneme

@reneme reneme commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

This adds overrides for allowed_signature_schemes and acceptable_signature_schemes in TLS::Text_Policy (addressing an API short-coming pointed out here). That way, users can specify exactly what signature schemes they wish to use. Before, users could only specify the allowed hash function and allowed signature algorithm. That made it impossible to restrict ECDSA to just the brainpool curves in TLS 1.3, for instance.

As a by-product this allows simplifying the RFC 8448 test somewhat by using this new policy feature.

@reneme
reneme requested a review from randombit July 29, 2026 10:53
@reneme reneme self-assigned this Jul 29, 2026
Copilot AI review requested due to automatic review settings July 29, 2026 10:53
@reneme reneme added the enhancement Enhancement or new feature label Jul 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@reneme
reneme force-pushed the rene/tls_text_policy_for_signature_schemes branch from 7659074 to 6b64c77 Compare July 29, 2026 10:58

@randombit randombit left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

lgtm

@reneme
reneme merged commit 03c33e9 into master Jul 29, 2026
92 of 93 checks passed
@reneme
reneme deleted the rene/tls_text_policy_for_signature_schemes branch July 29, 2026 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants