-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Update Multer #1563
Comments
So this is going to be much simpler is expressjs/multer#229 is merged, otherwise we'll have to change the public API significantly for application routes and pre-generate Admin UI routes that accept white-listed field names. Fingers crossed. |
This can be closed |
@mxstbr actually... multer is still pinned to an old version, this is still an active concern. Just lower priority than other issues in 0.4 |
I've picked this up at keystonejs/keystone#3794 and will make the changes there. |
This has been merged and released into 1.1.0, if this helps anything? cc/@bassjacob @JedWatson |
yep, the .any change should make this an out of the box fix. I'm going to have a look into their security recommendations and timebox that before just shipping to .any change. |
Closing as a duplicate of #4428, which will be addressed in Keystone |
Multer has been updated with breaking changes to the API. We need to review the changes and update the implementation in
lib/core/mount.js
accordingly.The text was updated successfully, but these errors were encountered: