expect quoting only on field beginning #16

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants

rkusa commented Mar 5, 2012

I think the following example:
random field content containing "some quoted text" without being quoted itself
shouldn't be handled as a quotedField.
(I personally need this tolerance because I have to parse user provided csv files ...)
What do you think?

I also updated the README.md node-formidable example (I needed these changes to got it work).

Ah and... thanks for ya-csv :-)

Owner

koles commented May 9, 2013

I don't want to make the parser too tolerant by default. If you still need this feature, how about making it configurable?

Sorry for the scandalously slow response.

rkusa commented Jun 6, 2013

Sounds legit, to make the parser not too tolerant by default. I don't need this feature anymore. So, it is okay for me if we just close this pull request. Thanks anyway

koles closed this Jun 6, 2013

jonlong commented Jan 27, 2017

Bump to pull in the README changes from this PR — the node-formidable example in the current README isn't functional without them

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