Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix grab #8

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants

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

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

Owner

chjj commented Oct 16, 2012

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.

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

Owner

chjj commented Feb 10, 2013

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

I guess.

@chjj chjj closed this in a4d4fc9 Feb 10, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment