New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Fix] Module Options #88
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Or, you can pass options directly | ||
|
||
```js{}[nuxt.config.js] | ||
export default defineNuxtConfig({ | ||
modules: [ | ||
[ | ||
'nuxt-security', | ||
{ | ||
requestSizeLimiter: { | ||
value: { | ||
maxRequestSizeInBytes: 3000000, | ||
maxUploadFileRequestInBytes: 9000000, | ||
}, | ||
route: '/upload-file' | ||
} | ||
// Other options | ||
} | ||
] | ||
] | ||
}) | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: could you also add it to the README.md? It would be beneficial to have the same docs here and there :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I have added to README.md.
Please, review my PR again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I approve and merge!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will release it as a part of the next release (0.11.0) in the upcoming days :)
Types of changes
Description
This PR is related to #87.
I've implemented options from the module setup.
Checklist: