fix(core): spread storage options

[vejja]

Closes #433

Types of changes

• Bug fix (a non-breaking change which fixes an issue)
• New feature (a non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

The Rate Limiter options need to be spread in order to support all the unstorage drivers.

Previously, only the options property could be provided to the underlying driver, because this driver has only one property with the name options.

With this PR, any property name can be provided under the options property.

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes (if not applicable, please state why)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
nuxt-security	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 21, 2024 10:39am

[Baroshem]

Thanks for working on it. Unfortunately it will be a breaking change but with a proper release notes, I think we can have it in 2.0.0-rc.1

[vejja]

I'm not entirely sure it's a breaking change, it could actually be a bug fix.
It's very difficult to understand how nitro sets the unstorage options, and then how unstorage forwards these options to the underlying drivers. I'll do some local tests and will let you know.

[Baroshem]

I think it will be a breaking change purely because the interface for how certain feature will change. Wwith this change, users that uset the previous version of storage options, would have it not working anymore.

But no worries about that. This is not a huge breaking change as long as we provide release notes and migration guide.

[vejja]

I think it will be a breaking change purely because the interface for how certain feature will change. Wwith this change, users that uset the previous version of storage options, would have it not working anymore.

Yes that's right

[vejja]

@Baroshem : Update after further review
The bug @numselli found out for the redis driver was also a problem for all other drivers. The options were actually never passed to the underlying driver.
I'm fixing the TypeScript definitions to clarify how the options must be passed and what values are available for each driver.

Marking as ready now

[Baroshem]

Thanks for finding it and fixing it @vejja 💚!

I can't wait for this release. it will be 🔥