diff options
author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2022-12-05 21:39:22 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-12-05 21:39:22 (GMT) |
commit | a726f747e659efed674db1ebf57218c20d8c0c39 (patch) | |
tree | 772cdbde80f74e5a42d36eb476100dab5bcbce25 /Lib/http/server.py | |
parent | f3a58cece6c7346355397a3e7848b0df38a0e20d (diff) | |
download | cpython-a726f747e659efed674db1ebf57218c20d8c0c39.zip cpython-a726f747e659efed674db1ebf57218c20d8c0c39.tar.gz cpython-a726f747e659efed674db1ebf57218c20d8c0c39.tar.bz2 |
gh-100001: Omit control characters in http.server stderr logs. (GH-100002)
Replace control characters in http.server.BaseHTTPRequestHandler.log_message with an escaped \xHH sequence to avoid causing problems for the terminal the output is printed to.
(cherry picked from commit d8ab0a4dfa48f881b4ac9ab857d2e9de42f72828)
Co-authored-by: Gregory P. Smith <greg@krypto.org>
Diffstat (limited to 'Lib/http/server.py')
-rw-r--r-- | Lib/http/server.py | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/Lib/http/server.py b/Lib/http/server.py index f2aeb65..9845b0f 100644 --- a/Lib/http/server.py +++ b/Lib/http/server.py @@ -93,6 +93,7 @@ import email.utils import html import http.client import io +import itertools import mimetypes import os import posixpath @@ -562,6 +563,10 @@ class BaseHTTPRequestHandler(socketserver.StreamRequestHandler): self.log_message(format, *args) + # https://en.wikipedia.org/wiki/List_of_Unicode_characters#Control_codes + _control_char_table = str.maketrans( + {c: fr'\x{c:02x}' for c in itertools.chain(range(0x20), range(0x7f,0xa0))}) + def log_message(self, format, *args): """Log an arbitrary message. @@ -577,12 +582,16 @@ class BaseHTTPRequestHandler(socketserver.StreamRequestHandler): The client ip and current date/time are prefixed to every message. + Unicode control characters are replaced with escaped hex + before writing the output to stderr. + """ + message = format % args sys.stderr.write("%s - - [%s] %s\n" % (self.address_string(), self.log_date_time_string(), - format%args)) + message.translate(self._control_char_table))) def version_string(self): """Return the server software version string.""" |