strapi-community/strapi-plugin-rest-cache

Plugin ignores the header in useHeaders option

Closed this issue · 2 comments

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?

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.

Thanks a lot for the effort, @Boegie19