Skip to content

nginx to 1.25.1, other updates, and update docs and configs. #113

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

Merged
merged 4 commits into from
Jun 27, 2023

Conversation

justdan6
Copy link

@justdan6 justdan6 commented Jun 17, 2023

nginx to 1.25.1 and using nginx mainline as opposed to the quic branch
alpine to 3.17
BoringSSL and NJS to the most recent commit
http2 directive used
update docs and configs

#110

@macbre macbre added the enhancement New feature or request label Jun 26, 2023
@macbre macbre self-requested a review June 26, 2023 11:45
@macbre
Copy link
Owner

macbre commented Jun 26, 2023

@justdan6 - thanks for this PR!

We just need to change the tests as they assume certain response headers to be present:

<p>It works!</p>
+ grep --fixed-strings '< HTTP/3 200' /tmp/h3
< HTTP/3 200
+ grep --fixed-strings --invert-match -i '< server: nginx' /tmp/h3
+ grep --fixed-strings '< alt-svc: h3-27=":8889"; ma=86400, h3-28=":8889"; ma=86400, h3-29=":8889"; ma=86400' /tmp/h3

@macbre
Copy link
Owner

macbre commented Jun 27, 2023

@justdan6 - thanks. Merging and will tag it later today.

@macbre macbre merged commit 7f52155 into macbre:master Jun 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants