search

chjj / parted

Streaming body parser for node.js.
MIT License
63 stars 14 forks source link

Fix grab #8

Closed ghost closed 11 years ago

ghost commented 12 years ago

In some cases the filename was broken. If it had a comma for example. Maybe you have a better way of doing it?

andrewrk commented 12 years ago

@oddmarthon-lende can you post an example request that causes this behavior?

chjj commented 12 years ago

If it had a comma for example.

I tend to think a filename like that would be surrounded by quotes (a filename with a comma not surrounded by quotes would create a lot of ambiguity as to what is the end of the field). We could change simply change the regex to account for this.

ghost commented 11 years ago

Isn't this module obsolete? Doesn't Express have this built in now?

chjj commented 11 years ago

Isn't this module obsolete? Doesn't Express have this built in now?

I guess.