Skip to content

fix(web): configure shared servers once before serving listeners - #454

Open
siavashs wants to merge 1 commit into
prometheus:masterfrom
siavashs:fix/configure-once-before-serve
Open

siavashs wants to merge 1 commit into
prometheus:masterfrom
siavashs:fix/configure-once-before-serve

Conversation

@siavashs

@siavashs siavashs commented Oct 6, 2026

Copy link
Copy Markdown

ServeMultiple calls Serve concurrently for each listener on the same http.Server.
Repeated setup can wrap the handler more than once and race when updating the handler and TLS configuration.

Configure middleware and TLS once before starting listener goroutines.
Preserve the public APIs and the existing wait-for-all return behavior.

ServeMultiple calls Serve concurrently for each listener on the same
http.Server. Repeated setup can wrap the handler more than once and
race when updating the handler and TLS configuration.

Configure middleware and TLS once before starting listener goroutines.
Preserve the public APIs and the existing wait-for-all return behavior.

Signed-off-by: Siavash Safi <siavash@cloudflare.com>
@siavashs
siavashs marked this pull request as ready for review October 6, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant