[PATCH v4 07/12] Static: add "index" option

Alejandro Colomar (man-pages) alx.manpages at gmail.com
Mon Feb 14 16:50:31 UTC 2022


On 2/14/22 12:33, Alejandro Colomar (man-pages) wrote:
> Hi, Andrei!
> 
> On 1/28/22 13:27, Andrei Zeliankou wrote:
>>> On 23 Dec 2021, at 19:25, Alejandro Colomar <alx.manpages at gmail.com> 
>>> wrote:
>>>
>>> This supports a new option "index", that configures a custom index
>>> file name that will be served when a directory is requested.  This
>>> initial support only allows a single fixed string.  An example:
>>>
>>> {
>>>     "share": "/www/data/static/$uri",
>>>     "index": "lookatthis.htm"
>>> }
>>>
> [...]
> 
>> Also noticed problems:
>>
>> 1) can't compile patch set with specified `--debug` option:
>>
>> src/nxt_http_static.c:264:23: error: no member named 'index' in 
>> 'nxt_http_static_conf_t'
>>      nxt_var_raw(conf->index, &idx);
>>                  ~~~~  ^
> 
> Oops, fixed.
> 
>>
>> 2) router crash when index is the empty array
> 
> I'm still investigating this one.
> 

Fixed too.  So the tests are still pending, and the question regarding 
the 301 for non-regular files too.

Cheers,

Alex

-- 
Alejandro Colomar
Linux man-pages comaintainer; http://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/



More information about the unit mailing list