[PATCH]add new http status code
simohayha.bobo at gmail.com
Wed May 9 11:58:49 UTC 2012
On Wed, May 9, 2012 at 7:07 PM, Maxim Dounin <mdounin at mdounin.ru> wrote:
> On Tue, May 08, 2012 at 11:24:33PM +0200, Manlio Perillo wrote:
> > Il 08/05/2012 21:39, Maxim Dounin ha scritto:
> > > [...]
> > >> Blew is patch(for Nginx trunk) to use 429 replace 400 when request
> > >> is too large,
> > >> and use 431 replace 503 when find too many request(limit_req module).
> > >> addition to add new http status code 511.
> > >
> > > There are no plans to replace returned status codes in near
> > > future, at least not before we have clear understanding of effects
> > > of new codes on behaviour of various clients.
> > >
> > > And in case of limit_req/limit_conn this might happen at all due
> > > to compatibility reasons: there are lots of configs out there
> > > which expect 503 to be returned.
> > >
> > For limit_req/limit_conn this should not be a problem, since the error
> > code to return can be made configurable.
> It can be, but I'm not sure there are benefits (and the default
> should be kept the same anyway). If one wants to return any
> specific code, error_page might be used to change the code.
> Moreover, use of 429 in case of non-client-specific limits (that
> is, even if you use $remote_addr for limiting as NATs are common,
> not even talking about resource-specific limits, e.g. based on
> $server_name) looks flawed for me from standards point of view, as
> the status should be really 5xx, not 4xx.
> Maxim Dounin
> nginx-devel mailing list
> nginx-devel at nginx.org
I got it, thanks for your replay.
do not fear to be eccentric in opinion, for every opinion now accepted was
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the nginx-devel