Skip to content

Allow dynamic TLS record sizing#13416

Open
bneradt wants to merge 1 commit into
apache:masterfrom
bneradt:allow-dynamic-tls-record-sizing
Open

Allow dynamic TLS record sizing#13416
bneradt wants to merge 1 commit into
apache:masterfrom
bneradt:allow-dynamic-tls-record-sizing

Conversation

@bneradt

@bneradt bneradt commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

The documented -1 value for proxy.config.ssl.max_record_size is
incorrectly rejected by records validation, leaving dynamic TLS record
sizing unreachable from records.yaml.

This widens the accepted range to include the dynamic sentinel,
clarifies the documented modes, and extends TLS wire-level coverage to
verify the small-to-large record transition.

Fixes: #13288

Copilot AI review requested due to automatic review settings July 21, 2026 18:56
@bneradt bneradt added this to the 11.0.0 milestone Jul 21, 2026
@bneradt bneradt self-assigned this Jul 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

The documented -1 value for proxy.config.ssl.max_record_size is
incorrectly rejected by records validation, leaving dynamic TLS record
sizing unreachable from records.yaml.

This widens the accepted range to include the dynamic sentinel,
clarifies the documented modes, and extends TLS wire-level coverage to
verify the small-to-large record transition.

Fixes: apache#13288
@bneradt
bneradt force-pushed the allow-dynamic-tls-record-sizing branch from fc754e3 to 0987314 Compare July 21, 2026 19:02
Copilot AI review requested due to automatic review settings July 21, 2026 19:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

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.

proxy.config.ssl.max_record_size rejects the documented value -1 (dynamic TLS record sizing)

2 participants