search

freesoftwarefactory / parse-multipart

A javascript/nodejs multipart/form-data parser which operates on raw data.
MIT License
48 stars 79 forks source link

Parse the 'name' field and return it #16

Open suenchunhui opened 5 years ago

suenchunhui commented 5 years ago

Hi, I did a small patch to parse and return the name field, and return it. Was useful for having multiple uploaded files which are differentiated by unique name attributes in the html input tag.

christiansalazar commented 5 years ago

Thanku very much :)

2018-08-02 2:23 GMT-04:00 suenchunhui notifications@github.com:

Hi, I did a small patch to parse and return the name field, and return it. Was useful for having multiple uploaded files which are differentiated by unique name attributes in the html input tag.

You can view, comment on, or merge this pull request online at:

https://github.com/freesoftwarefactory/parse-multipart/pull/16 Commit Summary

  • parse the 'name' field and return it in the result
  • small fix to unwrap object

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/freesoftwarefactory/parse-multipart/pull/16, or mute the thread https://github.com/notifications/unsubscribe-auth/ABk4g3gTs5_ZXds1YjGlXr1BT7ABPSfOks5uMpr-gaJpZM4Vrsx8 .

--

Cristian A. Salazar

Building Next Generation Apps with Serverless/Microservices, follow me: @AmazonAwsVideos

(+569) 9649-3840 | https://plus.google.com/+ChristianSalazar Santiago. Chile.

Bitbucket https://bitbucket.org/christiansalazarh/ | Github https://github.com/freesoftwarefactory | Stackoverflow http://stackoverflow.com/users/937815/christian | Google + https://plus.google.com/+ChristianSalazar | Blog http://trucosdeprogramacionmovil.blogspot.cl/ | Youtube http://www.youtube.com/c/ChristianSalazar | Npm (NodeJs) https://www.npmjs.com/~csalazar

rambabusaravanan commented 5 years ago

Thanks @suenchunhui, I'm too looking for this feature

@christiansalazar Can you please merge this

christiansalazar commented 5 years ago

hello folks. im having some problems about merging anything new because the current master branch is heavily used by my primary customer. so i dont have so much time available to do the necesary tests on the new commits proposed by you or others. im sorry about this. The best thing you can do is clone from any of the existing branches (which have some solutions to specific cases proposed by others). Let me know. :)

El dom., 20 de ene. de 2019 04:27, Andro Babu notifications@github.com escribió:

Thanks @suenchunhui https://github.com/suenchunhui, I'm too looking for this feature

@christiansalazar https://github.com/christiansalazar Can you please merge this

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/freesoftwarefactory/parse-multipart/pull/16#issuecomment-455843859, or mute the thread https://github.com/notifications/unsubscribe-auth/ABk4g9CuwGwWjcPIKSX1ggs0n3eITKI4ks5vFBpNgaJpZM4Vrsx8 .

rambabusaravanan commented 5 years ago

Okay. Since I'm in need of it, just forked and published for my personal use and using like

npm install @androbabu/multipart-parser