Opened 2 years ago

#1573 new enhancement

adding text/css to the default list for the charset_types directive

Reported by: Zhang-Junzhi@… Owned by:
Priority: minor Milestone:
Component: nginx-module Version: 1.13.x
Keywords: Cc:
uname -a: Linux zjz-PC 4.9.0-deepin13-amd64 #1 SMP PREEMPT Deepin 4.9.57-1 (2017-10-19) x86_64 GNU/Linux
nginx -V: nginx version: nginx/1.13.3
built with OpenSSL 1.1.0f 25 May 2017
TLS SNI support enabled
configure arguments: --with-cc-opt='-g -O2 -fdebug-prefix-map=/build/nginx-7EsFS1/nginx-1.13.3=. -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-7EsFS1/nginx-1.13.3/debian/modules/nginx-auth-pam --add-dynamic-module=/build/nginx-7EsFS1/nginx-1.13.3/debian/modules/nginx-dav-ext-module --add-dynamic-module=/build/nginx-7EsFS1/nginx-1.13.3/debian/modules/nginx-echo --add-dynamic-module=/build/nginx-7EsFS1/nginx-1.13.3/debian/modules/nginx-upstream-fair --add-dynamic-module=/build/nginx-7EsFS1/nginx-1.13.3/debian/modules/ngx_http_substitutions_filter_module

Description

I would like to suggest adding text/css to the default list for the charset_types directive.

Since the W3C is now introducing the custom element feature, so there may also be a growing trend of non-ASCII markups used in CSS files as well as HTML files, things like this will be seen in the future:

x-測
{
}

So the encoding of CSS files is no longer unimportant. Although adding @charset rule in the beginning of a CSS file works for HTML documents, but if the user directly visits non-ASCII CSS files, they probably encounter messy code. So I suggest adding text/css to the default list for the charset_types directive.

Change History (0)

Note: See TracTickets for help on using tickets.