Skip to content

Commit

Permalink
Handle Google App Engine not setting _tunnel_host (#1527)
Browse files Browse the repository at this point in the history
  • Loading branch information
jpevarnek authored and sethmlarson committed Jan 24, 2019
1 parent 791e9b4 commit 799f50d
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/urllib3/connection.py
Expand Up @@ -171,7 +171,8 @@ def _new_conn(self):

def _prepare_conn(self, conn):
self.sock = conn
if self._tunnel_host:
# Google App Engine's httplib does not define _tunnel_host
if getattr(self, '_tunnel_host', None):
# TODO: Fix tunnel so it doesn't depend on self.sock state.
self._tunnel()
# Mark this connection as not reusable
Expand Down Expand Up @@ -305,7 +306,8 @@ def connect(self):
conn = self._new_conn()
hostname = self.host

if self._tunnel_host:
# Google App Engine's httplib does not define _tunnel_host
if getattr(self, '_tunnel_host', None):
self.sock = conn
# Calls self._set_hostport(), so self.host is
# self._tunnel_host below.
Expand Down

0 comments on commit 799f50d

Please sign in to comment.