Change to explicitly encode

pull/24725/head
Lidi Zheng 4 years ago
parent e191e98f37
commit 0e275f3d2b
  1. 4
      src/python/grpcio_tests/tests/_runner.py
  2. 6
      tools/run_tests/python_utils/port_server.py

@ -59,7 +59,7 @@ class CaptureFile(object):
"""Get all output from the redirected-to file if it exists.""" """Get all output from the redirected-to file if it exists."""
if self._into_file: if self._into_file:
self._into_file.seek(0) self._into_file.seek(0)
return bytes(self._into_file.read(), 'ascii') return self._into_file.read().encode('ascii')
else: else:
return bytes() return bytes()
@ -80,7 +80,7 @@ class CaptureFile(object):
value (str): What to write to the original file. value (str): What to write to the original file.
""" """
if six.PY3 and not isinstance(value, six.binary_type): if six.PY3 and not isinstance(value, six.binary_type):
value = bytes(value, 'ascii') value = value.encode('ascii')
if self._saved_fd is None: if self._saved_fd is None:
os.write(self._redirect_fd, value) os.write(self._redirect_fd, value)
else: else:

@ -157,7 +157,7 @@ class Handler(BaseHTTPRequestHandler):
self.end_headers() self.end_headers()
p = allocate_port(self) p = allocate_port(self)
self.log_message('allocated port %d' % p) self.log_message('allocated port %d' % p)
self.wfile.write(bytes('%d' % p, 'ascii')) self.wfile.write(str(p).encode('ascii'))
elif self.path[0:6] == '/drop/': elif self.path[0:6] == '/drop/':
self.send_response(200) self.send_response(200)
self.send_header('Content-Type', 'text/plain') self.send_header('Content-Type', 'text/plain')
@ -177,7 +177,7 @@ class Handler(BaseHTTPRequestHandler):
self.send_response(200) self.send_response(200)
self.send_header('Content-Type', 'text/plain') self.send_header('Content-Type', 'text/plain')
self.end_headers() self.end_headers()
self.wfile.write(bytes('%d' % _MY_VERSION, 'ascii')) self.wfile.write(str(_MY_VERSION).encode('ascii'))
elif self.path == '/dump': elif self.path == '/dump':
# yaml module is not installed on Macs and Windows machines by default # yaml module is not installed on Macs and Windows machines by default
# so we import it lazily (/dump action is only used for debugging) # so we import it lazily (/dump action is only used for debugging)
@ -192,7 +192,7 @@ class Handler(BaseHTTPRequestHandler):
'in_use': dict((k, now - v) for k, v in in_use.items()) 'in_use': dict((k, now - v) for k, v in in_use.items())
}) })
mu.release() mu.release()
self.wfile.write(bytes(out, 'ascii')) self.wfile.write(out.encode('ascii'))
elif self.path == '/quitquitquit': elif self.path == '/quitquitquit':
self.send_response(200) self.send_response(200)
self.end_headers() self.end_headers()

Loading…
Cancel
Save