[RFC] [PATCH] Autoindex: support sorting using URL parameters

Maxim Dounin mdounin at mdounin.ru
Wed Jan 30 17:50:57 UTC 2013


Hello!

On Tue, Jan 29, 2013 at 06:49:46PM +0100, Peter Wu wrote:

[...]

> RFC: this patch only adds support for URL parameters. Before implementing the 
> actual clickable headers I would like to have some feedback. Do you actually
> want to add such a sorting feature to the standard autoindex module? Sorting
> has also been requested on http://forum.nginx.org/read.php?10,211728

Overrall I tend to think that it's bad idea to do autoindex more 
customizable.  I would rather like to see it producing XML and 
hence any customization being possible with XSLT filter.

[...]

> Parsing URL parameters is done in a separate function which is currently
> non-void. Since the return value is not used at the moment, it can also be make
> void. Thoughts?

You may want to use ngx_http_arg() function instead of reinventing 
the wheel.

-- 
Maxim Dounin
http://nginx.com/support.html



More information about the nginx-devel mailing list