Adding some more HTTP status constants

[msva]

Some usefull status constants + prerequisite for #397

[agentzh]

@msva Thank you for looking into this. Two issues in your patch though:

1. Will you just add the constants you actually need? I'd rather add such constants only on actual user demand because they add extra overhead.
2. Please avoid using the C99 single-line comment syntax (i.e., // xxx) which violates the nginx coding style :)

[msva]

Hi.

1. Not that I really _need_ them (I currently have to use integers in the code instead of that missed constants), just I looked in the NgX constants header and added here all of missing (in comparsion with it) values. Plus few "just usefull ones" (missed even there), like 426 and 429, for example. Plus non-standard ones, used by Cloudflare (520-524), but having usefull meaning.

But, yes, initially, my idea about making this patch, started from my personal need of 100, 204, 206 and 409 statuses in exact small project ;)
-- and thoughts, that most of that new constants I'd use in another one a bit later ;)

2. Okay. But will it be enough to just remove comments? Will it make patch accepatble, or do you insist that I reduced constants count? ;)

[agentzh]

@msva I insist that we should only add constants actually used :)

[msva]

I've removed comments and reduced constants count.
Now it is only that ones, using in real projects ;)

[agentzh]

@msva Thanks! I'll look into this :)

[sirupsen]

Would like to see this in too! We use/will use a few of those as well (HTTP_CONTINUE, HTTP_NO_CONTENT, HTTP_TOO_MANY_REQUESTS)

[agentzh]

This line breaks backward-compatibility.

[msva]

@agentzh ping? ;) Can you look into this once again? 😉

[agentzh]

@msva Sorry for the delay on my side. Will you update the document in the file doc/HttpLuaModule.wiki for this change? Thanks!

[msva]

I almost did it already, but should I also add comments like "first added in XXX release"? ;) If so — I need to know, which release will be next: 0.9.16 or rc2 ;)

[msva]

anyway, here is next proposition ;)

[agentzh]

@msva Cool, thanks!

[msva]

there was a little mistake (I confused HTTP_VERSION_NOT_SUPPORTED's code in the source (and wrote fine in wiki). Fixed now ;)

// Btw, I thinking about renaming it to HTTP_VERSION_UNSUPPORTED and also about adding CloudFlare's 520 Unknown Error

How do you think?

[agentzh]

This should be 101 instead of 102, right?

[msva]

yup. Fixed.

[msva]

ping? Should I drop the codes you've commented above, or it is not that necessarily ;)

[agentzh]

@msva Patch applied to git master. Thanks for your contribution and sorry for the long delay on my side.