don't set http_proxy environment variable, because it affects things we don't want it to

This commit is contained in:
Noah Levitt 2015-07-15 17:33:29 -07:00
parent 923cd98652
commit efa3cd6269

View File

@ -44,8 +44,10 @@ class BrozzlerWorker:
} }
if self._proxy_server: if self._proxy_server:
ydl_opts["proxy"] = "http://{}".format(self._proxy_server) ydl_opts["proxy"] = "http://{}".format(self._proxy_server)
# see https://github.com/rg3/youtube-dl/issues/6087 ## XXX (sometimes?) causes chrome debug websocket to go through
os.environ["http_proxy"] = "http://{}".format(self._proxy_server) ## proxy. Maybe not needed thanks to hls_prefer_native.
## # see https://github.com/rg3/youtube-dl/issues/6087
## os.environ["http_proxy"] = "http://{}".format(self._proxy_server)
self._ydl = youtube_dl.YoutubeDL(ydl_opts) self._ydl = youtube_dl.YoutubeDL(ydl_opts)
def _next_url(self, site): def _next_url(self, site):