@typescript-eslint/naming-convention: add HTTP headers to ignorelist

[jonahsnider]

HTTP headers and other common words (ex. POST or GET) should be added to an ignorelist so the camelcase validator doesn't trigger for them.

[sindresorhus]

Does the rule have this ability?

[jonahsnider]

You could add a RegExp to match words with the filter property. Messy but should work.

[sindresorhus]

I don't think that's a good idea. It's messy and it will also make non-header properties with the same name not be linted.

[EdJoPaTo]

I think keys from other libraries should be allowed in general. Other libraries have their defined keys in options and so on and I cant those keys from this side.

Because of this I disabled this rule for most of my projects because two of my most used libraries have keys that are not written in the same way xo wants them to be. This results in hundreds of errors which made me prefer to just disable the rule at all. Which is sad as I like the idea of enforcing variable style.

[maxpain]

Any updates on this?

[jonahsnider]

No, but luckily I've since changed my mind about header naming. Since Node.js & the HTTP spec say headers are case-insensitive I am now of the opinion that header names should instead be enforced to be totally lowercase.