-
Notifications
You must be signed in to change notification settings - Fork 105
Update handling of cache_control changed in nginx 1.23.0 (Fixes #96) #97
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 handling of cache_control changed in nginx 1.23.0 (Fixes #96) #97
Conversation
@hnakamur you can add v1.23.0 to .travis.yml. |
Add patch for nginx 1.23.0 by hnakamur · Pull Request #24 · openresty/no-pool-nginx is needed to be merged for travis nginx 1.23.0 to be successful. |
@hnakamur no-pool-nginx.patch has been merged. |
@hnakamur would you please retrigger the travis? |
@zhuizhuhaomeng I tried to retrigger the travis, but the "Trigger build" menu in "More options" dropdown at https://app.travis-ci.com/github/openresty/srcache-nginx-module/jobs/574418725 is disabled. |
I can not retrigger the travis also. I think you need to add a new commit , eg: add one space in the travis.yml. |
thanks god, I can retrigger the travis after I login travis-ci.com |
Hi @hnakamur thanks your contribution. I have approved this PR.
|
Thanks for reviewing and approving! |
@hnakamur t/private.t run failed. |
@hnakamur thank you for your contribution. |
This pull request fixes Nginx 1.23.0 header changes · Issue #96 · openresty/srcache-nginx-module.
See nginx/nginx@3aef1d6 for changes in nginx.