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

2022-11-02 Thread Roman Arutyunyan
Hi, On Tue, Nov 01, 2022 at 05:14:02PM +0300, Maxim Dounin wrote: > 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 a

Re: [PATCH] fix ngx_realpath() for win32

2022-11-02 Thread Maxim Dounin
Hello! On Wed, Nov 02, 2022 at 03:43:50PM +0900, Won-Kyu Park wrote: > > 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 >

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

2022-11-02 Thread Maxim Dounin
Hello! On Wed, Nov 02, 2022 at 05:06:25PM +0400, Roman Arutyunyan wrote: [...] > # HG changeset patch > # User Roman Arutyunyan > # Date 1667382376 -14400 > # Wed Nov 02 13:46:16 2022 +0400 > # Node ID dc5f16e6a243c15f58e2c6a62f7a83f536729174 > # Parent 81b4326daac70d6de70abbc3fe36d4f6e3d

[PATCH] SSI: ensure context of main request exists for subrequest using SSI

2022-11-02 Thread Ciel via nginx-devel
# HG changeset patch # User Ciel Zhao # Date 1667411069 -28800 # Thu Nov 03 01:44:29 2022 +0800 # Node ID 85141e004b5af89a9d443bc0084a34291193567a # Parent 1ae25660c0c76edef14121ca64362f28b9d57a70 SSI: ensure context of main request exists for subrequest using SSI As the SSI parser always u