search

fastify / fastify-rate-limit

A low overhead rate limiter for your routes
MIT License
504 stars 71 forks source link

fix: apply merged params when custom route config is provided #393

Closed gurgunday closed 1 week ago

gurgunday commented 1 week ago

We cannot use the global params when groupId is used, it just breaks the route configs — I think this was an oversight in the original PR, it had tests that tested custom route configs, but they were actually not correct as they didn't test different values

groupId should only affect the request id generation