Re: [PATCH] Add proxy_protocol option to mail listener

2017-08-08 Thread Maxim Dounin
Hello! On Mon, Aug 07, 2017 at 03:23:12PM +0200, Kees Bos wrote: > # HG changeset patch > # User Kees Bos > # Date 1500565189 0 > #  Thu Jul 20 15:39:49 2017 + > # Node ID 327f18e079b175b14277a23e75715f5feee34d69 > # Parent  

Re: [PATCH 12 of 14] Proxy: add HTTP/2 support

2017-08-08 Thread Maxim Dounin
Hello! On Tue, Jul 25, 2017 at 06:28:16PM -0700, Piotr Sikora wrote: > Hey Maxim, > > > There are serious concerns about this and with other patches. > > Most generic ones are outlined below. > > > > 1. Server-side HTTP/2 code complexity concerns. > > > > As per discussion with Valentin,

[njs] Added tag 0.1.12 for changeset c07b060396be

2017-08-08 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/f47b1eed6ed8 branches: changeset: 396:f47b1eed6ed8 user: Igor Sysoev date: Tue Aug 08 18:06:53 2017 +0300 description: Added tag 0.1.12 for changeset c07b060396be diffstat: .hgtags | 1 + 1 files changed, 1 insertions(+), 0

[njs] Version 0.1.12.

2017-08-08 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/c07b060396be branches: changeset: 395:c07b060396be user: Igor Sysoev date: Tue Aug 08 18:05:14 2017 +0300 description: Version 0.1.12. diffstat: CHANGES | 6 ++ Makefile | 2 +- 2 files changed, 7 insertions(+), 1

[njs] Removed unused field.

2017-08-08 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/bc306c37d037 branches: changeset: 392:bc306c37d037 user: Igor Sysoev date: Tue Aug 08 17:47:55 2017 +0300 description: Removed unused field. diffstat: njs/njs_parser.c| 12 njs/njs_parser.h

[njs] Style fix in Unicode header files and Perl scripts which

2017-08-08 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/49a8c7cfb121 branches: changeset: 393:49a8c7cfb121 user: Igor Sysoev date: Tue Aug 08 17:47:56 2017 +0300 description: Style fix in Unicode header files and Perl scripts which generate the files. diffstat:

[njs] Disabled njs_object_prop_alloc() inlining.

2017-08-08 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/bd7e5e51b6be branches: changeset: 394:bd7e5e51b6be user: Igor Sysoev date: Tue Aug 08 18:05:11 2017 +0300 description: Disabled njs_object_prop_alloc() inlining. diffstat: njs/njs_object.c | 2 +- 1 files changed, 1

[njs] Style fixes.

2017-08-08 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/e2baf506be4c branches: changeset: 391:e2baf506be4c user: Igor Sysoev date: Tue Aug 08 17:47:53 2017 +0300 description: Style fixes. diffstat: njs/njs_generator.c | 5 + njs/njs_variable.c | 3 --- 2 files changed, 1

[nginx] release-1.13.4 tag

2017-08-08 Thread Maxim Dounin
details: http://hg.nginx.org/nginx/rev/2f48ab272052 branches: changeset: 7084:2f48ab272052 user: Maxim Dounin date: Tue Aug 08 18:00:11 2017 +0300 description: release-1.13.4 tag diffstat: .hgtags | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diffs

[nginx] nginx-1.13.4-RELEASE

2017-08-08 Thread Maxim Dounin
details: http://hg.nginx.org/nginx/rev/bbc642c813c8 branches: changeset: 7083:bbc642c813c8 user: Maxim Dounin date: Tue Aug 08 18:00:11 2017 +0300 description: nginx-1.13.4-RELEASE diffstat: docs/xml/nginx/changes.xml | 65

[nginx] Updated PCRE used for win32 builds.

2017-08-08 Thread Maxim Dounin
details: http://hg.nginx.org/nginx/rev/74431027af26 branches: changeset: 7082:74431027af26 user: Maxim Dounin date: Tue Aug 08 15:21:10 2017 +0300 description: Updated PCRE used for win32 builds. diffstat: misc/GNUmakefile | 2 +- 1 files changed, 1

[nginx] Slab: fixed initialization on win32.

2017-08-08 Thread Ruslan Ermilov
details: http://hg.nginx.org/nginx/rev/69f9ee0342db branches: changeset: 7081:69f9ee0342db user: Ruslan Ermilov date: Tue Aug 08 15:16:01 2017 +0300 description: Slab: fixed initialization on win32. On Windows, a worker process does not call ngx_slab_init() from

Re: Supporting the Forwarded header (RFC 7239)

2017-08-08 Thread Maxim Dounin
Hello! On Tue, Aug 08, 2017 at 11:52:58AM +0300, Vasiliy Faronov wrote: > Hi Maxim, > > Thank you for your reply. > > > > I don't see how this is explicitly allowed by RFC 7239. > > In Section 4: > >A proxy MAY remove all "Forwarded" header fields from a request. Obviously enough,

Re: [PATCH] Add proxy_protocol option to mail listener

2017-08-08 Thread Wayde Nie
Thanks Kees and Maxim, for the patch and the discussion. A very helpful community! This looks like it's working well for my use case for a tcp465/"ssl on" mail proxy server block. My earlier issues appear to have (at least partly) been related to STARTTLS with a tcp587/"starttls only" block that