Plugin ignores the header in useHeaders option
Closed this issue · 2 comments
NotAhmedNasr commented
The plugin seems to ignore the headers specified in useHeaders
option based on case-sensitivity. The problem is that the logic inside the filter method that checks for the header in the request doesn't use toLowerCase
on the key, although it does exactly that when reading the actual header value in the next line.
Is this intended or is it just a mistake?
Boegie19 commented
good catch ctx.request.header will always be the in full lowercease. aka everyting in useHeaders is lowercease it works else you are correct and it does not function correctly. let me open a PR to fix this.
NotAhmedNasr commented
Thanks a lot for the effort, @Boegie19