-
-
Notifications
You must be signed in to change notification settings - Fork 538
dietpi-letsencrypt: update HTTP/2 syntax and add HTTP/3/QUIC #7860
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
MichaIng
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This directive appeared with Nginx 1.25.1, hence is not available on Debian Bookworm and below yet. On Trixie, we might want to add the quic parameter instead. Weirdly there is a http3 directive, which is however enabled by default, but without QUIC it seems, and the http2 directive is still off by default. So inconsistent. So:
- Up to Bookworm: Keep things as they are
- From Trixie on: Replace
http2argument withquic, and addhttp2directive instead.
|
I did a bit of testing and IDK if http3 is it worth yet. see https://nginx-wiki.getpagespeed.com/config/http3/#full-production-configuration But the biggest hurdle would be, that the user would need to forward port 443 on UDP too. I doubt anybody is aware of that except they really want QUIC for some specific reason. |
|
Oh it needs a dedicated |
Updated Nginx configuration to support HTTP/2 and QUIC.
remove newline to pass shellcheck
daa84b5 to
dc4b7c5
Compare
Apply "http2" parameter on Bookworm, and new "http2" directive as well as HTTP/3 with QUIC from Trixie on, where it is supported. Also apply each directive separately, always preserving any existing matching directive.
|
Old vs new way is now applied depending on Debian version, and I split each of them into a separate It applies correctly on a manually prepared Syntax test only expectedly fails on either first of @JappeHallunken you escaped the |
Yes, you’re right, only the single quotes need to be escaped. |
Nginx throws a warning at startup, because
Changed from
to