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.

Sign up for GitHub

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

Jump to bottom

fix escaping fields #152

Open

ycintre wants to merge 2 commits into asafdav:master from ycintre:fix-escaping

ycintre commented Mar 16, 2016

Escape the field with double quotes if it contains a double quote in the value.

This is specified in the RFC as follows:

6 . Fields containing line breaks (CRLF), double quotes, and commas
should be enclosed in double-quotes.

ycintre added 2 commits

March 16, 2016 13:18


          fix escaping if quote is present in field

660d7dd


          fix escaping when a custom field delimiter is specified

b54cbb9

Author

ycintre commented Mar 16, 2016

I also added a fix to the escape mechanism to use the user-specified field delimiter instead of hard-coded comma

ycintre changed the title ~~fix escaping if quote is present in field~~ fix escaping fields

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

Labels

None yet