diff options
author | Maximilian Hils <git@maximilianhils.com> | 2016-05-25 20:11:34 -0700 |
---|---|---|
committer | Maximilian Hils <git@maximilianhils.com> | 2016-05-25 20:11:34 -0700 |
commit | d3477e27fa4ffbcfa9c1b9aa937d1d54448cc597 (patch) | |
tree | 37e4eb1be1c0887279192c61487bb3079ae458c1 /netlib/utils.py | |
parent | 7a8da48a306dfc8e43239d7f2a141c465e40ab77 (diff) | |
download | mitmproxy-d3477e27fa4ffbcfa9c1b9aa937d1d54448cc597.tar.gz mitmproxy-d3477e27fa4ffbcfa9c1b9aa937d1d54448cc597.tar.bz2 mitmproxy-d3477e27fa4ffbcfa9c1b9aa937d1d54448cc597.zip |
bytes_to_escaped_str: always escape single quotes
Diffstat (limited to 'netlib/utils.py')
-rw-r--r-- | netlib/utils.py | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/netlib/utils.py b/netlib/utils.py index fe11cb5b..73b2adb3 100644 --- a/netlib/utils.py +++ b/netlib/utils.py @@ -425,6 +425,10 @@ def safe_subn(pattern, repl, target, *args, **kwargs): def bytes_to_escaped_str(data): """ Take bytes and return a safe string that can be displayed to the user. + + Single quotes are always escaped, double quotes are never escaped: + "'" + bytes_to_escaped_str(...) + "'" + gives a valid Python string. """ # TODO: We may want to support multi-byte characters without escaping them. # One way to do would be calling .decode("utf8", "backslashreplace") first @@ -432,7 +436,9 @@ def bytes_to_escaped_str(data): if not isinstance(data, bytes): raise ValueError("data must be bytes") - return repr(data).lstrip("b")[1:-1] + # We always insert a double-quote here so that we get a single-quoted string back + # https://stackoverflow.com/questions/29019340/why-does-python-use-different-quotes-for-representing-strings-depending-on-their + return repr('"' + data).lstrip("b")[2:-1] def escaped_str_to_bytes(data): |