Use lower case header can improve 3x performance

krakenjs / lusca

Application security for express apps.

Other

1.78k stars 123 forks source link

Use lower case header can improve 3x performance #91

Closed fengmk2 closed 7 years ago

fengmk2 commented 7 years ago

see https://gist.github.com/fengmk2/1634043ec1d40e09b53197b817411131

linkRace commented 7 years ago

Hey @fengmk2 sorry for the delay, trying to take over maintaining this. I'm assuming tests passed and whatnot? Seems like a simple change otherwise, will want another +1 from my team.

grawk commented 7 years ago

LGTM. :+1:

linkRace commented 7 years ago

@indutny @shaunwarman @gabrielcsapo another +1 before merge?

shaunwarman commented 7 years ago

👍

gabrielcsapo commented 7 years ago

👍

grawk commented 7 years ago

👍