Fix semgrep warnings

Author: Vitaliy Saveliev

.semgrepignore

@@ -0,0 +1,2 @@
+env.example.bat
+env.example.sh

selvpcclient/resources/tokens.py

@@ -46,8 +46,6 @@ def delete_many(self, token_ids, raise_if_not_found=True):
         for token_id in token_ids:
             try:
                 self.delete(token_id)
-                log.info("Token %s has been deleted", token_id)
             except ClientException as err:
                 if raise_if_not_found:
                     raise err
-                log.error("%s %s", err, token_id)

selvpcclient/util.py

@@ -211,7 +211,7 @@ def make_curl(url, method, data):
             v = str()
             if value:
                 v = value.encode('utf-8')
-            h = hashlib.sha1(v)
+            h = hashlib.sha256(v)
             d = h.hexdigest()
             value = "{SHA1}%s" % d
         header = ' -H "%s: %s"' % (key, value)
@@ -225,15 +225,17 @@ def make_curl(url, method, data):
 def is_url(data):
     """Checks if getting value is valid url and path exists."""
     try:
-        r = requests.head(data)
-    except Exception:
+        r = requests.head(data, timeout=15)
+        r.raise_for_status()
+    except requests.RequestException:
         return False
     return r.status_code == requests.codes.ok
 
 
 def process_logo_by_url(url):
     """Download and encode image by url."""
-    res = requests.get(url)
+    res = requests.get(url, timeout=15)
+    res.raise_for_status()
     encoded_logo = base64.b64encode(res.content)
     return encoded_logo
 

tests/cli/__init__.py

@@ -1,12 +1,13 @@
 import json
 
-import mock
+from unittest import mock
+
 from selvpcclient.client import Client
 from selvpcclient.shell import CLI
 
 
 def prepare_to_run_command(cmd):
-    pass
+    pass  # nosemgrep: python.lang.best-practice.pass-body.pass-body-fn
 
 
 class FakeStdout(object):

tests/rest/__init__.py

@@ -1,6 +1,5 @@
-import mock
-
 from datetime import datetime, timedelta
+from unittest import mock
 
 from selvpcclient.httpclient import HTTPClient, RegionalHTTPClient
 

tests/test_util.py

@@ -97,7 +97,7 @@ def function_that_takes_theme_params(logo=None, color=""):
 def test_process_theme_params_invalid_logo():
     @process_theme_params
     def function_that_takes_theme_params(logo=None, color=''):
-        pass
+        pass  # nosemgrep: python.lang.best-practice.pass-body.pass-body-fn
 
     with pytest.raises(Exception):
         function_that_takes_theme_params(logo='is \' not path or url!!!',
@@ -107,7 +107,7 @@ def function_that_takes_theme_params(logo=None, color=''):
 def test_process_theme_params_wrong_path():
     @process_theme_params
     def function_that_takes_theme_params(logo=None, color=''):
-        pass
+        pass  # nosemgrep: python.lang.best-practice.pass-body.pass-body-fn
 
     with pytest.raises(Exception):
         function_that_takes_theme_params(logo='/wrong/path/logo.jpg',