Opened 6 years ago

Last modified 2 years ago

#458 accepted defect

Win32: autoindex module doesn't support Unicode names

Reported by: kroward@… Owned by:
Priority: minor Milestone:
Component: nginx-core Version:
Keywords: win32 Cc:
uname -a: Microsoft Windows XP [Version 5.1.2600]
nginx -V: nginx/1.5.8 (current nginx trunk)

Description

Functions for traversing directories use ANSI versions of FindFirstFile?() and FindNextFile?(), so any characters in filenames besides basic latin become broken.

Proposed patch fix this issue converting WCHAR names to utf-8.

Attachments (1)

win32_read_dir.patch (5.9 KB) - added by kroward@… 6 years ago.
Win32: WCHAR support it ngx_read_dir() to enable Unicode in autoindex

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by kroward@…

Win32: WCHAR support it ngx_read_dir() to enable Unicode in autoindex

comment:1 Changed 6 years ago by mdounin

  • Status changed from new to accepted

Ack, it needs to be fixed. The patch in question needs more work. (And please also consider reading http://nginx.org/en/docs/contributing_changes.html, thanks.)

comment:2 Changed 6 years ago by kroward@…

I forgot to mention than behavior of autoindex module depends on selected charset. The patch does work with 'charset utf-8' directive in config file, but probably will give inconsistent results with other settings.

comment:3 Changed 2 years ago by mdounin

  • Keywords win32 added

See also #1433.

Note: See TracTickets for help on using tickets.