-
Notifications
You must be signed in to change notification settings - Fork 260
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
Crashes when the multipart request is empty. And crashes the node instance. #6
Comments
@mbisono I'm trying to reproduce - can you show me your express request handler code? |
Hi, this was my route handler definition: router.post('/', expressFileupload(), require('./set-logo')); I think it crashed before entering the 'set-logo' handler. Hard to tell since it's been a while. I only used it for a couple of days. It worked fine and uploaded the file if I selected one using Advanced Rest Client. But If left the file selection box empty the route would crash. |
I'm having a similar problem when using multipart with |
Can be replicated by launching a rest client like Advanced REST client and trying to upload a file without selecting one.
6:57:30 PM web.1 | events.js:85
6:57:30 PM web.1 | throw er; // Unhandled 'error' event
6:57:30 PM web.1 | ^
6:57:30 PM web.1 | Error: Unexpected end of multipart data
6:57:30 PM web.1 | at /Users/xxxxxx/Documents/workspace2/node-tickets/node_modules/dicer/lib/Dicer.js:62:28
6:57:30 PM web.1 | at process._tickCallback (node.js:355:11)
The text was updated successfully, but these errors were encountered: