search

repsheet / repsheet-nginx

The nginx module for Repsheet
Apache License 2.0
82 stars 11 forks source link

Multi header #30

Closed prokopowicz closed 8 years ago

prokopowicz commented 8 years ago

Dave - this PR includes the earlier work we did, and cleans it up a little.

The pattern I used for allocating space is as simple and primitive as can be, like Gilligan's Island. It doesn't require any data structures or malloc.

abedra commented 8 years ago

Can we talk more about the reason behind wanting multiple header names? I understand different values, but I don't understand the push for multiple header names. It makes consumption of the header names conditional and a lot easier to get wrong for consumers. I'm open to additional interpretations on this but I don't currently understand why it would be a good idea for the header name itself to change.