Opened 9 years ago

Closed 6 years ago

#9 closed enhancement (wontfix)

autoindex_max_name_len directive

Reported by: shipilev.livejournal.com Owned by: somebody
Priority: minor Milestone:
Component: nginx-module Version: 1.1.x
Keywords: autoindex Cc:
uname -a: Linux persephone 2.6.32-33-generic-pae #72-Ubuntu SMP Fri Jul 29 22:06:29 UTC 2011 i686 GNU/Linux
nginx -V: nginx: nginx version: nginx/1.1.1
nginx: built by gcc 4.4.3 (Ubuntu 4.4.3-4ubuntu5)
nginx: configure arguments:

Description

Current nginx autoindex module has hardcoded NGX_HTTP_AUTOINDEX_NAME_LEN constant. It makes sense to let user control it: nowadays, 50 characters are not enough to represent full file names. Moreover, I suspect UTF-8 names are getting twice lower limit of 25 characters.

Attachments (2)

ng-autoindex-namelen-1.patch (3.5 KB ) - added by shipilev.livejournal.com 9 years ago.
proposed patch
ng-autoindex-namelen-fixed.patch (3.8 KB ) - added by Bryan Whitehead 7 years ago.
Updated patch for nginx-1.6.0

Download all attachments as: .zip

Change History (6)

by shipilev.livejournal.com, 9 years ago

proposed patch

comment:1 by Aleksey Shipilev, 8 years ago

Any updates? Any problems with the proposal? Any problems with the patch?

by Bryan Whitehead, 7 years ago

Updated patch for nginx-1.6.0

comment:2 by Bryan Whitehead, 7 years ago

Original patch was for nginx 1.1.1, this is the same patch just for a recent version: 1.6.0

This is a feature I was looking for and it would be great if it was added.

Thanks!

comment:3 by Domenick Petrella, 6 years ago

I had to move a web server off of nginx because of this, unfortunately.

comment:4 by Valentin V. Bartenev, 6 years ago

Resolution: wontfix
sensitive: 0
Status: newclosed

There are no plans to enhance HTML representation of autoindex. Instead now you can use XML or JSON(P) output formats where no such limitation exists (see 33c08d7e2915 and 99751fe3bc3b).

And using the XSLT module you can transform the XML into HTML the way you like more.

Note: See TracTickets for help on using tickets.