[nginx] Free nginx: changed server name.

Daniel K. d at ja.vu
Tue Feb 20 16:22:57 UTC 2024


On 2/20/24 14:09, Maxim Dounin wrote:
> diff --git a/src/http/v2/ngx_http_v2_filter_module.c b/src/http/v2/ngx_http_v2_filter_module.c
> --- a/src/http/v2/ngx_http_v2_filter_module.c
> +++ b/src/http/v2/ngx_http_v2_filter_module.c
> @@ -115,12 +115,14 @@ ngx_http_v2_header_filter(ngx_http_reque
>      ngx_http_core_srv_conf_t  *cscf;
>      u_char                     addr[NGX_SOCKADDR_STRLEN];
>  
> -    static const u_char nginx[5] = "\x84\xaa\x63\x55\xe7";
>  #if (NGX_HTTP_GZIP)
>      static const u_char accept_encoding[12] =
>          "\x8b\x84\x84\x2d\x69\x5b\x05\x44\x3c\x86\xaa\x6f";
>  #endif
>  
> +    static size_t nginx_name_len = ngx_http_v2_literal_size(NGINX_NAME);
> +    static u_char nginx_name[ngx_http_v2_literal_size(NGINX_VER)];

It certainly works, but we could save a few bytes by using
NGINX_NAME/NGINX_NAME above...


>      static size_t nginx_ver_len = ngx_http_v2_literal_size(NGINX_VER);
>      static u_char nginx_ver[ngx_http_v2_literal_size(NGINX_VER)];

... just as we have NGINX_VER/NGINX_VER here.


Daniel K.




More information about the nginx-devel mailing list