Fix semgrep warnings

Author: Vitaliy Saveliev

.github/workflows/secure.yml

@@ -5,6 +5,9 @@ on: push
 jobs:
   # Sample GitHub Actions:
   # https://semgrep.dev/docs/semgrep-ci/sample-ci-configs#sample-github-actions-configuration-file
+  #
+  # CLI Reference:
+  # https://semgrep.dev/docs/cli-reference
   semgrep:
     runs-on: ubuntu-latest
     container:
@@ -14,7 +17,7 @@ jobs:
       security-events: write
     steps:
       - uses: actions/checkout@v4
-      - run: semgrep scan --sarif --output=semgrep.sarif --error
+      - run: semgrep scan --sarif --output=semgrep.sarif --error --severity=WARNING
         env:
           SEMGREP_RULES: >-
             p/bandit
@@ -54,7 +57,7 @@ jobs:
           format: 'sarif'
           output: 'trivy.sarif'
           exit-code: '1'
-          severity: 'CRITICAL,HIGH'
+          severity: 'MEDIUM,CRITICAL,HIGH'
       - uses: github/codeql-action/upload-sarif@v3
         with:
           sarif_file: trivy.sarif

.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,11 +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):
+    # nosemgrep
     pass
 
 

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,6 +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=''):
+        # nosemgrep
         pass
 
     with pytest.raises(Exception):
@@ -107,6 +108,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=''):
+        # nosemgrep
         pass
 
     with pytest.raises(Exception):