From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter =?utf-8?q?M=C3=BCller?= To: development@lists.ipfire.org Subject: [PATCH v2] fireinfo: support upstream proxy with authentication Date: Mon, 29 Oct 2018 18:22:10 +0100 Message-ID: <20181029172210.4157-1-peter.mueller@link38.eu> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2808004207553000283==" List-Id: --===============2808004207553000283== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Fireinfo could not send its profile to https://fireinfo.ipfire.org/ if the machine is behind an upstream proxy which requires username and password. This is fixed by tweaking urllib2's opening handler. To apply this on existing installations, the fireinfo package needs to be shipped during an update. The second version of this patch fixes bogus indention, assembles proxy authentication string more readable and preserves HTTP proxy handler. Fixes #11905 Signed-off-by: Peter M=C3=BCller Cc: Michael Tremer --- src/sendprofile | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) mode change 100644 =3D> 100755 src/sendprofile diff --git a/src/sendprofile b/src/sendprofile old mode 100644 new mode 100755 index b836567..1f32440 --- a/src/sendprofile +++ b/src/sendprofile @@ -73,10 +73,21 @@ def send_profile(profile): request.add_header("User-Agent", "fireinfo/%s" % fireinfo.__version__) =20 # Set upstream proxy if we have one. - # XXX this cannot handle authentication proxy =3D get_upstream_proxy() + if proxy["host"]: - request.set_proxy(proxy["host"], "http") + # handling upstream proxies with authentication is more tricky... + if proxy["user"] and proxy["pass"]: + prx_auth_string =3D "http://%s:%s@%s/" % (proxy["user"], proxy["pass"], p= roxy["host"]) + + proxy_handler =3D urllib2.ProxyHandler({'http': prx_auth_string}) + proxy_handler =3D urllib2.ProxyHandler({'https': prx_auth_string}) + auth =3D urllib2.HTTPBasicAuthHandler() + opener =3D urllib2.build_opener(proxy_handler, auth, urllib2.HTTPHandler) + urllib2.install_opener(opener) + else: + request.set_proxy(proxy["host"], "http") + request.set_proxy(proxy["host"], "https") =20 try: urllib2.urlopen(request, timeout=3D60) --=20 2.16.4 --===============2808004207553000283==--