Skip to content

Conversation

@lieryan
Copy link

@lieryan lieryan commented Jan 21, 2015

According to Section 3.2.3:

Senders SHOULD NOT escape octets in quoted-strings that do not require escaping (i.e., other than DQUOTE and the backslash octet).

However, currently quote_string() escapes SEPARATOR characters that does not need to be escaped:

>>> print quote_string('test/one two\\ @three')
"test\/one\ two\\\ \@three"

They rather should be:

>>> print quote_string('test/one two\\ @three')
"test/one two\\ @three"

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants