-
-
Notifications
You must be signed in to change notification settings - Fork 878
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
Add two more lingering_ config options #1435
Conversation
nginx::config is a classBreaking changes to this file MAY impact these 2 modules (near match):nginx is a classBreaking changes to this file WILL impact these 14 modules (exact match):Breaking changes to this file MAY impact these 34 modules (near match):
This module is declared in 9 of 576 indexed public
|
The two failures for CentOS 6 seem to be due to the CentOS 6 base repo going away (http://mirror.centos.org/centos-6/6/readme) |
Once CI goes green on #1436 this PR can be rebased and testing here should also be good. |
2073b09
to
96c9d28
Compare
See https://nginx.org/en/docs/http/ngx_http_core_module.html\#lingering_timeout tighten types for lingering_close list the 3 valid options as enum for lingering_time enforece a non empty string
96c9d28
to
e43790b
Compare
Pull Request (PR) description
Adding two more lingering_ config options
lingering_close
lingering_time
See https://nginx.org/en/docs/http/ngx_http_core_module.html#lingering_timeout
This Pull Request (PR) fixes the following issues
n/a