Opened 2 weeks ago

Closed 2 weeks ago

Last modified 12 days ago

#1681 closed defect (wontfix)

Default_type

Reported by: OlafvdSpek@… Owned by:
Priority: minor Milestone:
Component: other Version: 1.15.x
Keywords: Cc:
uname -a: Linux unstable 4.18.0-2-amd64 #1 SMP Debian 4.18.10-2 (2018-11-02) x86_64 GNU/Linux
nginx -V: nginx version: nginx/1.14.1 built with OpenSSL 1.1.1 11 Sep 2018 (running with OpenSSL 1.1.1a 20 Nov 2018) TLS SNI support enabled configure arguments: --with-cc-opt='-g -O2 -fdebug-prefix-map=/build/nginx-DqItAQ/nginx-1.14.1=. -fstack-protector-strong -Wformat -Werror=format-security -fPIC -Wdate-time -D_FORTIFY_SOURCE=2' --with-ld-opt='-Wl,-z,relro -Wl,-z,now -fPIC' --prefix=/usr/share/nginx --conf-path=/etc/nginx/nginx.conf --http-log-path=/var/log/nginx/access.log --error-log-path=/var/log/nginx/error.log --lock-path=/var/lock/nginx.lock --pid-path=/run/nginx.pid --modules-path=/usr/lib/nginx/modules --http-client-body-temp-path=/var/lib/nginx/body --http-fastcgi-temp-path=/var/lib/nginx/fastcgi --http-proxy-temp-path=/var/lib/nginx/proxy --http-scgi-temp-path=/var/lib/nginx/scgi --http-uwsgi-temp-path=/var/lib/nginx/uwsgi --with-debug --with-pcre-jit --with-http_ssl_module --with-http_stub_status_module --with-http_realip_module --with-http_auth_request_module --with-http_v2_module --with-http_dav_module --with-http_slice_module --with-threads --with-http_addition_module --with-http_geoip_module=dynamic --with-http_gunzip_module --with-http_gzip_static_module --with-http_image_filter_module=dynamic --with-http_sub_module --with-http_xslt_module=dynamic --with-stream=dynamic --with-stream_ssl_module --with-stream_ssl_preread_module --with-mail=dynamic --with-mail_ssl_module --add-dynamic-module=/build/nginx-DqItAQ/nginx-1.14.1/debian/modules/http-auth-pam --add-dynamic-module=/build/nginx-DqItAQ/nginx-1.14.1/debian/modules/http-dav-ext --add-dynamic-module=/build/nginx-DqItAQ/nginx-1.14.1/debian/modules/http-echo --add-dynamic-module=/build/nginx-DqItAQ/nginx-1.14.1/debian/modules/http-upstream-fair --add-dynamic-module=/build/nginx-DqItAQ/nginx-1.14.1/debian/modules/http-subs-filter

Description

docs:
Default: default_type text/plain;

nginx.conf: default_type application/octet-stream;

  1. IMO the stock conf file shouldn't deviate from the default.
  2. I think the default is wrong, if nginx doesn't know the type it shouldn't send the type.

Change History (9)

comment:1 Changed 2 weeks ago by OlafvdSpek@…

Same for keepalive_timeout.

comment:2 Changed 2 weeks ago by mdounin

  • Resolution set to wontfix
  • Status changed from new to closed

Thank you for your suggestion. You can use default_type applicatin/octet-stream; in your configuration if you think it is better thing to use, and this is what actually example configuration as shipped with nginx does right after defining common types:

http {
    include       mime.types;
    default_type  application/octet-stream;

While application/octet-stream might be a safer option, using text/plain by default is also a quite reasonable choice, given that to make application/octet-stream usable in practice you have to define common types anyway. I don't think we have to change the default.

As for the keepalive_timeout, the default is 75s, and this is what most installations would normally use unless there are good reasons to use other values.

comment:3 Changed 2 weeks ago by OlafvdSpek@…

and this is what most installations would normally use

Then why does the 'example' nginx.conf contain these lines?

#keepalive_timeout 0;
keepalive_timeout 65;

comment:4 Changed 2 weeks ago by mdounin

Because one of the goals of the example configuration is to show how to tune various important parameters.

comment:5 Changed 2 weeks ago by OlafvdSpek@…

So that file isn't supposed to be used as the stock conf for nginx?

comment:6 Changed 2 weeks ago by mdounin

It is. And one of the goals of any stock conf is to show how to tune various important parameters.

comment:7 Changed 2 weeks ago by OlafvdSpek@…

Having two different nginx defaults is confusing..

comment:8 follow-up: Changed 12 days ago by OlafvdSpek@…

As for the keepalive_timeout, the default is 75s, and this is what most installations would normally use unless there are good reasons to use other values.

This isn't actually true right, if most installations are using the stock nginx.conf?

comment:9 in reply to: ↑ 8 Changed 12 days ago by mdounin

This isn't actually true right, if most installations are using the stock nginx.conf?

The example value as in the stock nginx.conf is practically the same.

Note: See TracTickets for help on using tickets.