Skip to content

reverse_proxy: apply keep-alive setting for h2c requests#6343

Merged
mholt merged 1 commit intomasterfrom
h2c-proxy-protocol
Aug 8, 2024
Merged

reverse_proxy: apply keep-alive setting for h2c requests#6343
mholt merged 1 commit intomasterfrom
h2c-proxy-protocol

Conversation

@WeidiDeng
Copy link
Member

Fix 6342.

@WeidiDeng WeidiDeng changed the title disable keep alive for h2c requests reverse_proxy: disable keep alive for h2c requests May 27, 2024
@WeidiDeng WeidiDeng changed the title reverse_proxy: disable keep alive for h2c requests reverse_proxy: apply keep-alive setting for h2c requests May 31, 2024
@mholt
Copy link
Member

mholt commented Jun 2, 2024

Thanks Weidi! As I understand it, this depends on an upstream patch first?

@WeidiDeng
Copy link
Member Author

Yes, or else h2c will stop working if keep alive is disabled. For now it's buggy behaviour, but it works 🤷 .

@WeidiDeng
Copy link
Member Author

Just wait for the release of golang/x/net. It's fixed in the master.

@mholt
Copy link
Member

mholt commented Jun 13, 2024

Great! Will do 😄

@mholt mholt merged commit 21af88f into master Aug 8, 2024
@mholt mholt deleted the h2c-proxy-protocol branch August 8, 2024 12:53
@mholt
Copy link
Member

mholt commented Aug 8, 2024

Great - thanks for the ping! And the patch :)

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.

Using proxy_protocol v2 with h2c backend gives wrong IP address to backend.

2 participants