From efa3cd62696c8fa6acffacfd9f218256b77d9708 Mon Sep 17 00:00:00 2001 From: Noah Levitt Date: Wed, 15 Jul 2015 17:33:29 -0700 Subject: [PATCH] don't set http_proxy environment variable, because it affects things we don't want it to --- brozzler/worker.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/brozzler/worker.py b/brozzler/worker.py index 75ac8ba..52e383a 100755 --- a/brozzler/worker.py +++ b/brozzler/worker.py @@ -44,8 +44,10 @@ class BrozzlerWorker: } if self._proxy_server: ydl_opts["proxy"] = "http://{}".format(self._proxy_server) - # see https://github.com/rg3/youtube-dl/issues/6087 - os.environ["http_proxy"] = "http://{}".format(self._proxy_server) + ## XXX (sometimes?) causes chrome debug websocket to go through + ## 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) def _next_url(self, site):