-
-
Notifications
You must be signed in to change notification settings - Fork 131
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 and test for invalid operations
and map
field types
#139
Conversation
src/processRequest.mjs
Outdated
@@ -193,6 +197,14 @@ export const processRequest = ( | |||
createError(413, `${maxFiles} max file uploads exceeded.`) | |||
) | |||
|
|||
if (typeof operations !== 'object') |
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'm not sure this check is necessary, because operations
has already been checked at this point?
Still need to handle an invalid ‘map’ type.
Thanks for bringing this up! I've handled a few issues (e.g. I'll take it from here, as the |
`null` is more likely to reveal issues as it is technically an object.
operations
and map
field types.
operations
and map
field types.operations
and map
field types
Thanks! This at least solved one of the problems I had, but null is another tricky one indeed! What would be the ETA for this to be in a new release? |
@MickVanDuijn I plan to release today. |
Thanks 🙌 |
Published in v8.0.5 🚀 |
🚀 Thanks for the quick release! |
When
operations
is of typestring
, andoperationsPath.set
is called, the error was not correctly propagated.