[nginx] Upstream: moved header initializations to separate funct...

2014-11-19 Thread Roman Arutyunyan
details: http://hg.nginx.org/nginx/rev/195561ef367f branches: changeset: 5907:195561ef367f user: Roman Arutyunyan date: Wed Nov 19 17:33:21 2014 +0300 description: Upstream: moved header initializations to separate functions. No functional changes. diffstat: src/http/modules/ngx_

[nginx] Upstream: moved header lists to separate structures.

2014-11-19 Thread Roman Arutyunyan
details: http://hg.nginx.org/nginx/rev/f8e80f8c7fc7 branches: changeset: 5908:f8e80f8c7fc7 user: Roman Arutyunyan date: Wed Nov 19 17:33:22 2014 +0300 description: Upstream: moved header lists to separate structures. No functional changes. diffstat: src/http/modules/ngx_http_fast

[nginx] Proxy: renamed and rearranged fields in proxy configurat...

2014-11-19 Thread Roman Arutyunyan
details: http://hg.nginx.org/nginx/rev/29fa5023bd6f branches: changeset: 5910:29fa5023bd6f user: Roman Arutyunyan date: Wed Nov 19 17:33:24 2014 +0300 description: Proxy: renamed and rearranged fields in proxy configuration. No functional changes. diffstat: src/http/modules/ngx_h

[nginx] Scgi: do not push redundant NULL element into conf->params.

2014-11-19 Thread Roman Arutyunyan
details: http://hg.nginx.org/nginx/rev/548f704c1907 branches: changeset: 5906:548f704c1907 user: Roman Arutyunyan date: Wed Nov 19 17:33:21 2014 +0300 description: Scgi: do not push redundant NULL element into conf->params. diffstat: src/http/modules/ngx_http_scgi_module.c | 7 --

[nginx] Upstream: different header lists for cached and uncached...

2014-11-19 Thread Roman Arutyunyan
details: http://hg.nginx.org/nginx/rev/8d0cf26ce071 branches: changeset: 5909:8d0cf26ce071 user: Roman Arutyunyan date: Wed Nov 19 17:33:23 2014 +0300 description: Upstream: different header lists for cached and uncached requests. The upstream modules remove and alter a number of cl

Re: [PATCH] Not Modified: prefer entity tags over date validators

2014-11-19 Thread Maxim Dounin
Hello! On Tue, Nov 18, 2014 at 05:15:46PM -0800, Piotr Sikora wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1416359232 28800 > # Tue Nov 18 17:07:12 2014 -0800 > # Node ID 3efade6bb02f7962a5120e1a1f95a1dc8f0b6a4c > # Parent 2f7e557eab5b501ba71418febd3de9ef1c0ab4f1 > Not Modi

Re: [PATCH 2 of 2] Cache: send conditional requests only for cached 200 OK responses

2014-11-19 Thread Maxim Dounin
Hello! On Tue, Nov 18, 2014 at 05:15:49PM -0800, Piotr Sikora wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1416359233 28800 > # Tue Nov 18 17:07:13 2014 -0800 > # Node ID 16f4ca8391ddd98ba99b00a46c0b56390f38e0a2 > # Parent 99e65578bc80960b2fdf494e048678dd97bba029 > Cache: s

Re: [PATCH 1 of 2] Cache: remove unused valid_msec fields

2014-11-19 Thread Maxim Dounin
Hello! On Tue, Nov 18, 2014 at 05:15:48PM -0800, Piotr Sikora wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1416359233 28800 > # Tue Nov 18 17:07:13 2014 -0800 > # Node ID 99e65578bc80960b2fdf494e048678dd97bba029 > # Parent 2f7e557eab5b501ba71418febd3de9ef1c0ab4f1 > Cache: r

[nginx] Cache: add support for Cache-Control's s-maxage response...

2014-11-19 Thread Maxim Dounin
details: http://hg.nginx.org/nginx/rev/88d55e5934f7 branches: changeset: 5911:88d55e5934f7 user: Piotr Sikora date: Tue Nov 18 17:07:14 2014 -0800 description: Cache: add support for Cache-Control's s-maxage response directive. Signed-off-by: Piotr Sikora diffstat: src/http/ngx_

Re: [PATCH] Cache: add support for Cache-Control's s-maxage response directive

2014-11-19 Thread Maxim Dounin
Hello! On Tue, Nov 18, 2014 at 05:15:51PM -0800, Piotr Sikora wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1416359234 28800 > # Tue Nov 18 17:07:14 2014 -0800 > # Node ID b7b345ad11d81cbf6c17e5933aad9ce3af4f16c8 > # Parent 2f7e557eab5b501ba71418febd3de9ef1c0ab4f1 > Cache: a

[nginx] Refactored ngx_solaris_sendfilev_chain().

2014-11-19 Thread Valentin Bartenev
details: http://hg.nginx.org/nginx/rev/4dd67e5d958e branches: changeset: 5914:4dd67e5d958e user: Valentin Bartenev date: Wed Nov 19 21:17:11 2014 +0300 description: Refactored ngx_solaris_sendfilev_chain(). Though ngx_solaris_sendfilev_chain() shouldn't suffer from the problem menti

[nginx] Moved the code for coalescing file buffers to a separate...

2014-11-19 Thread Valentin Bartenev
details: http://hg.nginx.org/nginx/rev/ac3f78219f85 branches: changeset: 5915:ac3f78219f85 user: Valentin Bartenev date: Wed Aug 13 15:11:45 2014 +0400 description: Moved the code for coalescing file buffers to a separate function. diffstat: src/core/ngx_buf.c

[nginx] Moved writev() handling code to a separate function.

2014-11-19 Thread Valentin Bartenev
details: http://hg.nginx.org/nginx/rev/2c64b69daec5 branches: changeset: 5917:2c64b69daec5 user: Valentin Bartenev date: Wed Aug 13 15:11:45 2014 +0400 description: Moved writev() handling code to a separate function. This reduces code duplication and unifies debug logging of the wr

[nginx] Merged conditions in the ngx_*_sendfile_chain() functions.

2014-11-19 Thread Valentin Bartenev
details: http://hg.nginx.org/nginx/rev/e044893b4587 branches: changeset: 5916:e044893b4587 user: Valentin Bartenev date: Wed Aug 13 15:11:45 2014 +0400 description: Merged conditions in the ngx_*_sendfile_chain() functions. No functional changes. diffstat: src/os/unix/ngx_darwin_

[nginx] Introduced the ngx_output_chain_to_iovec() function.

2014-11-19 Thread Valentin Bartenev
details: http://hg.nginx.org/nginx/rev/8e903522c17a branches: changeset: 5913:8e903522c17a user: Valentin Bartenev date: Tue Oct 07 11:38:57 2014 +0400 description: Introduced the ngx_output_chain_to_iovec() function. It deduplicates code of the send chain functions and uses only pr

[nginx] Renamed ngx_handle_sent_chain() to ngx_chain_update_sent().

2014-11-19 Thread Valentin Bartenev
details: http://hg.nginx.org/nginx/rev/de68ed551bfb branches: changeset: 5912:de68ed551bfb user: Valentin Bartenev date: Wed Nov 19 21:16:19 2014 +0300 description: Renamed ngx_handle_sent_chain() to ngx_chain_update_sent(). No functional changes. diffstat: src/core/ngx_buf.c

[nginx] Fixed type of sendfile() return value on Linux.

2014-11-19 Thread Valentin Bartenev
details: http://hg.nginx.org/nginx/rev/c50b5ed3cd4b branches: changeset: 5918:c50b5ed3cd4b user: Valentin Bartenev date: Wed Nov 19 21:18:13 2014 +0300 description: Fixed type of sendfile() return value on Linux. There was no real problem since the amount of bytes can be sent is lim

[nginx] Style.

2014-11-19 Thread Valentin Bartenev
details: http://hg.nginx.org/nginx/rev/fddc6bed1e6e branches: changeset: 5919:fddc6bed1e6e user: Valentin Bartenev date: Wed Nov 19 21:46:01 2014 +0300 description: Style. diffstat: src/os/unix/ngx_darwin_sendfile_chain.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)

Re: [PATCH] Not Modified: prefer entity tags over date validators

2014-11-19 Thread Piotr Sikora
Hey Maxim, > Current nginx behaviour is to respect both, and I don't see real > reasons to change the behaviour. How about adhering to RFC standards? RFC7232 clearly describes the precedence for evaluation of conditional requests (section 6) and puts even more emphasis on it by saying that If-(Un

Re: [PATCH 1 of 2] Cache: remove unused valid_msec fields

2014-11-19 Thread Piotr Sikora
Hey Maxim, > I would rather not, for two reasons: > > - I would actually like to see support for subsecond cache > validity times added eventually (unlikely to happen in the near > future, but still). That's valid reason, but it's going to be of rather limited use, because neither "Expires" n

Re: [PATCH 2 of 2] Cache: send conditional requests only for cached 200 OK responses

2014-11-19 Thread Piotr Sikora
Hey Maxim, > And what's wrong with it? > I don't see why this should be status-specific. Because, according to the RFC, 304 Not Modified can be only emitted for 200 OK responses, so we know that revalidation of other status codes cannot happen, at least not when talking with well-behaving server,