Skip to content
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

Update Nginx patches to version 1.11.6 #53

Open
zawn opened this issue Nov 21, 2016 · 9 comments
Open

Update Nginx patches to version 1.11.6 #53

zawn opened this issue Nov 21, 2016 · 9 comments

Comments

@zawn
Copy link

zawn commented Nov 21, 2016

Update Nginx patches to version 1.11.6

@injust
Copy link
Contributor

injust commented Nov 21, 2016

All 3 patches (dynamic TLS records, SPDY, and ChaCha20-Poly1305) that work for nginx 1.11.5 work for 1.11.6 already.

@zawn
Copy link
Author

zawn commented Nov 21, 2016

[root@localhost nginx]# patch -p1 < ../nginx__1.11.5_dynamic_tls_records.patch
patching file src/event/ngx_event_openssl.c
patching file src/event/ngx_event_openssl.h
patching file src/http/modules/ngx_http_ssl_module.c
Hunk #2 succeeded at 574 (offset 6 lines).
Hunk #3 succeeded at 644 (offset 6 lines).
Hunk #4 succeeded at 838 (offset 6 lines).
patching file src/http/modules/ngx_http_ssl_module.h

There is a text line offset, but can be used

@lenovouser
Copy link

@injust nope, the SPDY patch doesn't work for the 1.11.6

@injust
Copy link
Contributor

injust commented Nov 23, 2016

@lenovouser
Copy link

@lenovouser
Copy link

#36 (comment)

@amirymk
Copy link

amirymk commented Mar 22, 2017

Hello,

There is a problem with theis patch when it is applied to nginx version 1.11.11
[root@server nginx-1.11.11]# patch -p1 < ./nginx-spdy-1.11.5+.patch patching file auto/modules
Hunk #1 FAILED at 139.
Hunk #2 FAILED at 171.
Hunk #3 FAILED at 232.
Hunk #4 FAILED at 440.
4 out of 4 hunks FAILED -- saving rejects to file auto/modules.rej
patching file auto/options

Please modify the patch and fix these errors .

@injust
Copy link
Contributor

injust commented Mar 22, 2017

Added new versions in #63

@injust
Copy link
Contributor

injust commented Oct 17, 2017

@zawn Can this be closed yet?

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

No branches or pull requests

4 participants