[PATCH] fix ngx_realpath() for win32

2022-11-01 Thread Won-Kyu Park
# HG changeset patch # User Won-Kyu Park # Date 1667298839 -32400 # Tue Nov 01 19:33:59 2022 +0900 # Node ID c964b33d60679f5cd8d43a8f859d16c2ac6f89e1 # Parent 1ae25660c0c76edef14121ca64362f28b9d57a70 fix ngx_realpath() for win32 diff -r 1ae25660c0c7 -r c964b33d6067 src/os/win32/ngx_files.c

Re: [PATCH] Core: support for reading PROXY protocol v2 TLVs

2022-11-01 Thread Maxim Dounin
Hello! On Mon, Oct 31, 2022 at 04:07:00PM +0400, Roman Arutyunyan wrote: > While testing the feature, it became clear that 107 bytes as maximum PROXY > protocol header size may not be enough. This limit came from v1 and stayed > unchanged when v2 was added. With this limit, there are only 79 by

Re: [PATCH] fix ngx_realpath() for win32

2022-11-01 Thread Maxim Dounin
Hello! On Tue, Nov 01, 2022 at 08:15:48PM +0900, Won-Kyu Park wrote: > # HG changeset patch > # User Won-Kyu Park > # Date 1667298839 -32400 > # Tue Nov 01 19:33:59 2022 +0900 > # Node ID c964b33d60679f5cd8d43a8f859d16c2ac6f89e1 > # Parent 1ae25660c0c76edef14121ca64362f28b9d57a70 > fix ngx

Re: [PATCH] fix ngx_realpath() for win32

2022-11-01 Thread Won-Kyu Park
> Hello! > > On Tue, Nov 01, 2022 at 08:15:48PM +0900, Won-Kyu Park wrote: > > ... > > Could you please clarify "as expected"? > > It doesn't look like _fullpath() resolves symbolic links, which is expected > behaviour for $realpath_root. > You are right but for most situations, translating a rel