Skip to content

Commit c42c372

Browse files
committed
[http,pushsafer] Remove dependency on future package
1 parent 340ca51 commit c42c372

File tree

2 files changed

+27
-18
lines changed

2 files changed

+27
-18
lines changed

mqttwarn/services/http_urllib.py

+14-9
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,15 @@
55
__copyright__ = 'Copyright 2014 Ben Jones'
66
__license__ = 'Eclipse Public License - v 1.0 (http://www.eclipse.org/legal/epl-v10.html)'
77

8-
from future import standard_library
9-
standard_library.install_aliases()
8+
try:
9+
from urllib.parse import urlparse, urlencode, urljoin
10+
from urllib.request import urlopen, Request
11+
from urllib.error import HTTPError
12+
except ImportError:
13+
from urlparse import urlparse # type: ignore[no-redef]
14+
from urllib import urlencode, urljoin # type: ignore[no-redef,attr-defined]
15+
from urllib2 import urlopen, Request, HTTPError # type: ignore[no-redef]
1016

11-
import urllib.request, urllib.parse, urllib.error
1217
import base64
1318

1419
try:
@@ -71,18 +76,18 @@ def plugin(srv, item):
7176
resource = url
7277
if not resource.endswith('?'):
7378
resource = resource + '?'
74-
resource = resource + urllib.parse.urlencode(params)
79+
resource = resource + urlencode(params)
7580
else:
7681
resource = url
7782

78-
request = urllib.request.Request(resource)
83+
request = Request(resource)
7984

8085
if srv.SCRIPTNAME is not None:
8186
request.add_header('User-agent', srv.SCRIPTNAME)
8287
if basicauth_token is not None:
8388
request.add_header("Authorization", "Basic %s" % basicauth_token)
8489

85-
resp = urllib.request.urlopen(request, timeout=timeout)
90+
resp = urlopen(request, timeout=timeout)
8691
data = resp.read()
8792
#srv.logging.debug("HTTP response:\n%s" % data)
8893
except Exception as e:
@@ -93,13 +98,13 @@ def plugin(srv, item):
9398

9499
if method.upper() == 'POST':
95100
try:
96-
request = urllib.request.Request(url)
101+
request = Request(url)
97102
if params is not None:
98103
if tojson is not None:
99104
encoded_params = json.dumps(params)
100105
request.add_header('Content-Type', 'application/json')
101106
else:
102-
encoded_params = urllib.parse.urlencode(params)
107+
encoded_params = urlencode(params)
103108
else:
104109
if tojson is not None:
105110
encoded_params = item.payload
@@ -116,7 +121,7 @@ def plugin(srv, item):
116121
request.add_header("Authorization", "Basic %s" % basicauth_token)
117122

118123
srv.logging.debug("before send")
119-
resp = urllib.request.urlopen(request, timeout=timeout)
124+
resp = urlopen(request, timeout=timeout)
120125
data = resp.read()
121126
#srv.logging.debug("HTTP response:\n%s" % data)
122127
except Exception as e:

mqttwarn/services/pushsafer.py

+13-9
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,17 @@
88
import dataclasses
99
from collections import OrderedDict
1010

11-
from future import standard_library
12-
standard_library.install_aliases()
1311
import os
1412
import json
15-
import urllib.request, urllib.parse, urllib.error
16-
import urllib.request, urllib.error, urllib.parse
17-
import urllib.parse
13+
14+
try:
15+
from urllib.parse import urlparse, urlencode, urljoin
16+
from urllib.request import urlopen, Request
17+
from urllib.error import HTTPError
18+
except ImportError:
19+
from urlparse import urlparse # type: ignore[no-redef]
20+
from urllib import urlencode, urljoin # type: ignore[no-redef,attr-defined]
21+
from urllib2 import urlopen, Request, HTTPError # type: ignore[no-redef]
1822

1923
import typing as t
2024

@@ -39,10 +43,10 @@ def pushsafer(**kwargs):
3943
# Don't submit empty parameters to Pushsafer.
4044
filter_empty_parameters(kwargs)
4145

42-
url = urllib.parse.urljoin(PUSHSAFER_API, "api")
43-
data = urllib.parse.urlencode(kwargs).encode('utf-8')
44-
req = urllib.request.Request(url, data)
45-
response = urllib.request.urlopen(req, timeout=3)
46+
url = urljoin(PUSHSAFER_API, "api")
47+
data = urlencode(kwargs).encode('utf-8')
48+
req = Request(url, data)
49+
response = urlopen(req, timeout=3)
4650
output = response.read()
4751
data = json.loads(output)
4852

0 commit comments

Comments
 (0)