~emersion/public-inbox

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
2 2

[PATCH tlstunnel] Add HTTP/2 note

minus
Details
Message ID
<20201106153436.2122897-1-minus@mnus.de>
DKIM signature
fail
Download raw message
Patch: +5 -0 DKIM signature: fail
---
 tlstunnel.1.scd | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/tlstunnel.1.scd b/tlstunnel.1.scd
index b692ee9..345429e 100644
--- a/tlstunnel.1.scd
+++ b/tlstunnel.1.scd
@@ -78,6 +78,11 @@ The following directives are supported:
		The email address to use when creating or selecting an existing ACME
		server account

# NOTES

HTTP/2 requires using the PROXY protocol and the backend must support PROXY
v2's ALPN TLV, since HTTP/2 is negociated that way.

# FILES

_/etc/tlstunnel/config_
-- 
2.29.2

[tlstunnel/patches/.build.yml] build failed

builds.sr.ht
Details
Message ID
<C6WAHQ5KM9K3.3P7NMFBQH2RSD@cirno>
In-Reply-To
<20201106153436.2122897-1-minus@mnus.de> (view parent)
DKIM signature
missing
Download raw message
tlstunnel/patches/.build.yml: FAILED in 35s

[Add HTTP/2 note][0] from [minus][1]

[0]: https://lists.sr.ht/~emersion/public-inbox/patches/14756
[1]: mailto:minus@mnus.de

✗ #336054 FAILED tlstunnel/patches/.build.yml https://builds.sr.ht/~emersion/job/336054
Details
Message ID
<YuM_BOzGXc6inxaER0YoYT47F2xVLYP-Q27zX7QNtBq7f2bmGy_lN-6DIsXRTOg5BLgCNEAjFbjgd0lbbDu9K_JJ8RTsahBpddTCH8yrQ9k=@emersion.fr>
In-Reply-To
<20201106153436.2122897-1-minus@mnus.de> (view parent)
DKIM signature
fail
Download raw message
DKIM signature: fail
On Friday, November 6, 2020 4:34 PM, minus <minus@mnus.de> wrote:

> +# NOTES
> +
> +HTTP/2 requires using the PROXY protocol and the backend must support PROXY
> +v2's ALPN TLV, since HTTP/2 is negociated that way.

Even with PROXY enabled and ALPN TLVs picked up by the backend, it's
not enough to enable HTTP/2. tlstunnel also needs to advertise HTTP/2
in the TLS ALPN extension, and that work isn't merged yet [1] (although
some WIP patches are available).

I think it's best to defer merging this note until we support changing
the list of protocols advertised via ALPN.

[1]: https://todo.sr.ht/~emersion/tlstunnel/11
Reply to thread Export thread (mbox)