Skip to content

Commit 860fa62

Browse files
committed
remove urlparse
Signed-off-by: Varsha GS <[email protected]>
1 parent daffb67 commit 860fa62

File tree

7 files changed

+10
-50
lines changed

7 files changed

+10
-50
lines changed

example/autoprofile/app.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
import time
55
import threading
66
import random
7-
import traceback
87
import sys
98
import os
109

@@ -13,14 +12,6 @@
1312
os.environ['INSTANA_AUTOPROFILE'] = 'yes'
1413
import instana
1514

16-
try:
17-
# python 2
18-
from urllib2 import urlopen
19-
except ImportError:
20-
# python 3
21-
from urllib.request import urlopen
22-
23-
2415
# Simulate CPU intensive work
2516
def simulate_cpu():
2617
for i in range(5000000):

instana/instrumentation/celery/hooks.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33

44

55
import opentracing
6+
67
from ...log import logger
78
from ...singletons import tracer
89
from ...util.traceutils import get_active_tracer
@@ -12,11 +13,7 @@
1213
from celery import registry, signals
1314
from .catalog import task_catalog_get, task_catalog_pop, task_catalog_push, get_task_id
1415

15-
try:
16-
from urllib import parse
17-
except ImportError:
18-
import urlparse as parse
19-
import urllib
16+
from urllib import parse
2017

2118

2219
def add_broker_tags(span, broker_url):

instana/instrumentation/django/middleware.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,7 @@ def process_exception(self, request, exception):
108108

109109
def __url_pattern_route(self, view_name):
110110
from django.conf import settings
111-
try:
112-
from django.urls import RegexURLPattern as URLPattern
113-
from django.urls import RegexURLResolver as URLResolver
114-
except ImportError:
115-
from django.urls import URLPattern, URLResolver
111+
from django.urls import RegexURLResolver as URLResolver
116112

117113
urlconf = __import__(settings.ROOT_URLCONF, {}, {}, [''])
118114

instana/instrumentation/google/cloud/collectors.py

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,7 @@
22
# (c) Copyright Instana Inc. 2020
33

44
import re
5-
6-
try:
7-
# Python 3
8-
from urllib.parse import unquote
9-
except ImportError:
10-
# Python 2
11-
from urllib import unquote
5+
from urllib.parse import unquote
126

137
# _storage_api defines a conversion of Google Storage JSON API requests into span tags as follows:
148
# request_method -> path_matcher -> collector

instana/util/__init__.py

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,10 @@
33

44
import json
55
import time
6-
76
from collections import defaultdict
8-
import pkg_resources
7+
from urllib import parse
98

10-
try:
11-
from urllib import parse
12-
except ImportError:
13-
import urlparse as parse
14-
import urllib
9+
import pkg_resources
1510

1611
from ..log import logger
1712

instana/util/ids.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,6 @@
1010

1111
BAD_ID = "BADCAFFE" # Bad Caffe
1212

13-
string_types = str
14-
1513

1614
def generate_id():
1715
""" Generate a 64bit base 16 ID for use as a Span or Trace ID """
@@ -41,7 +39,7 @@ def header_to_long_id(header):
4139
if isinstance(header, bytes):
4240
header = header.decode('utf-8')
4341

44-
if not isinstance(header, string_types):
42+
if not isinstance(header, str):
4543
return BAD_ID
4644

4745
try:
@@ -70,7 +68,7 @@ def header_to_id(header):
7068
if isinstance(header, bytes):
7169
header = header.decode('utf-8')
7270

73-
if not isinstance(header, string_types):
71+
if not isinstance(header, str):
7472
return BAD_ID
7573

7674
try:

instana/util/secrets.py

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2,16 +2,8 @@
22
# (c) Copyright Instana Inc. 2020
33

44
import re
5-
import re
6-
import sys
7-
8-
try:
9-
from urllib import parse
10-
except ImportError:
11-
import urlparse as parse
12-
import urllib
5+
from urllib import parse
136

14-
from ..util import PY2, PY3
157
from ..log import logger
168

179

@@ -127,10 +119,7 @@ def strip_secrets_from_query(qp, matcher, kwlist):
127119
logger.debug("strip_secrets_from_query: unknown matcher")
128120
return qp
129121

130-
if PY2:
131-
result = urllib.urlencode(params, doseq=True)
132-
else:
133-
result = parse.urlencode(params, doseq=True)
122+
result = parse.urlencode(params, doseq=True)
134123
query = parse.unquote(result)
135124

136125
if path:

0 commit comments

Comments
 (0)