Skip to content

Fix HTTP flow control policy reload - #13695

Open
bneradt wants to merge 1 commit into
apache:masterfrom
bneradt:reload-http2-flow-control-policies
Open

bneradt wants to merge 1 commit into
apache:masterfrom
bneradt:reload-http2-flow-control-policies

Conversation

@bneradt

@bneradt bneradt commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

HTTP/2 flow-control changes were reported as reloadable but had no
effect because connections kept startup-only policy values. The reload
callbacks also retained pointers to expired stack variables.

This patch validates startup and runtime policy values and publishes
them through persistent atomic storage for both connection directions.
The regression coverage checks policy transitions on fresh connections
through live updates and records.yaml reloads.

Fixes: #13694

Co-authored-by: Codex Astra Medium

Copilot AI lite review requested due to automatic review settings September 15, 2026 23:03
@bneradt bneradt added this to the 11.0.0 milestone Sep 15, 2026
@bneradt bneradt self-assigned this Sep 15, 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.

HTTP/2 flow-control changes were reported as reloadable but had no
effect because connections kept startup-only policy values. The reload
callbacks also retained pointers to expired stack variables.

This patch validates startup and runtime policy values and publishes
them through persistent atomic storage for both connection directions.
The regression coverage checks policy transitions on fresh connections
through live updates and records.yaml reloads.

Fixes: apache#13694

Co-authored-by: Codex Astra Medium
@bneradt
bneradt force-pushed the reload-http2-flow-control-policies branch from e9904f1 to f09c345 Compare September 15, 2026 23:09
@bneradt bneradt changed the title Reload HTTP/2 flow-control policies safely Fix HTTP flow control policy reload Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

HTTP/2 flow-control policies do not reload despite RECU_DYNAMIC

2 participants