[PATCH] Contrib: vim syntax, update 3rd party module directives

2017-02-28 Thread othree
# HG changeset patch # User othree # Date 1488339792 -28800 # Wed Mar 01 11:43:12 2017 +0800 # Node ID b57cf9a765d8f5603f1bf359eb95dadc0a832f18 # Parent 8b7fd958c59f8280d167fe7dd93f1942bfed5876 Contrib: vim syntax, update 3rd party module directives. diff --git a/contrib/vim/syntax/nginx.vi

[PATCH 2 of 3] Upstream: allow recovery from "429 Too Many Requests" response

2017-02-28 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1488324535 28800 # Tue Feb 28 15:28:55 2017 -0800 # Node ID e21f12a958010e1f3e5cdc1640859e335e032ca5 # Parent 9a63d6e990d230db0ec6b03250265447f648526e Upstream: allow recovery from "429 Too Many Requests" response. This change adds "http_429"

[PATCH 3 of 3] Limit req: change default response code when rate-limiting

2017-02-28 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1488324535 28800 # Tue Feb 28 15:28:55 2017 -0800 # Node ID c9d43c652ac776068e78f695dde00606eed184f8 # Parent e21f12a958010e1f3e5cdc1640859e335e032ca5 Limit req: change default response code when rate-limiting. Previously, "503 Service Unavail

[PATCH 1 of 3] HTTP: add support for "429 Too Many Requests" response (RFC6585)

2017-02-28 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1488324535 28800 # Tue Feb 28 15:28:55 2017 -0800 # Node ID 9a63d6e990d230db0ec6b03250265447f648526e # Parent 8b7fd958c59f8280d167fe7dd93f1942bfed5876 HTTP: add support for "429 Too Many Requests" response (RFC6585). This change adds reason ph

RE: Add missing static specifiers

2017-02-28 Thread Eran Kornblau
Thanks for the review! Updated patch attached, comments inline below > Hello! > > > # HG changeset patch > > # User Eran Kornblau # Date 1488300547 18000 > > # Tue Feb 28 11:49:07 2017 -0500 > > # Node ID 4b4b8f5413a4a1679d6ad0aa444e29e3f55b6b2a > > # Parent 8b7fd958c59f8280d167fe7dd93f19

Re: Add missing static specifiers

2017-02-28 Thread Maxim Dounin
Hello! On Tue, Feb 28, 2017 at 07:20:49PM +, Eran Kornblau wrote: > Hi all, > > Wrote a small script to find missing 'static's in my module - > https://github.com/kaltura/nginx-vod-module/blob/master/test/test_static.py > > I executed it on nginx core and found a few of those too, see attac

Add missing static specifiers

2017-02-28 Thread Eran Kornblau
Hi all, Wrote a small script to find missing 'static's in my module - https://github.com/kaltura/nginx-vod-module/blob/master/test/test_static.py I executed it on nginx core and found a few of those too, see attached patch. The logic for finding these was - 1. An exported symbol (found by