Merge pull request #8 from CMCC-Foundation/no-shell

Low and Medium severity bandit reports handling
CMCC-Foundation · Dec 11, 2024 · b9204fd · b9204fd
2 parents de30697 + 726ccf9
commit b9204fd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 116 deletions.
diff --git a/src/model/RUN/MODEL_SRC/ReadSatData_EMSA.py b/src/model/RUN/MODEL_SRC/ReadSatData_EMSA.py
diff --git a/src/utils/utils.py b/src/utils/utils.py
@@ -416,13 +416,13 @@ def parallel_processing_mrc(concat, exp_folder=None):
     def run_process_gebco(gebco, grid, output_dir):
         script_name = "scripts/pre_processing/preproc_gebco_mdk2.py"
         # Run the external Python script as a subprocess
-        subprocess.run([f"{sys.executable}", script_name, gebco, grid, output_dir])
+        subprocess.run([f"{sys.executable}", script_name, gebco, grid, output_dir]) # nosec
 
     @staticmethod
     def run_process_gshhs(gshhs, grid, output_dir):
         script_name = "scripts/pre_processing/preproc_gshhs_mdk2.py"
         # Run the external Python script as a subprocess
-        subprocess.run([f"{sys.executable}", script_name, gshhs, grid, output_dir])
+        subprocess.run([f"{sys.executable}", script_name, gshhs, grid, output_dir]) # nosec
 
     def rename_netcdf_variables_mdk3(ds):
         """
@@ -506,5 +506,8 @@ def oil_volume_shapefile(config, dens=0.922, thick=0.00001):
     print(Utils.set_product("mercator_daily"))
     result = Utils.compute_domain(120, 33, 41, 35, 10)
     result2 = Utils.compute_domain(480, 33, 41, 35, 10)
-    assert result2 == result
+
+    if result2 != result:
+        raise AssertionError(f"Validation failed: result2 ({result2}) != result ({result})")
+
     print(result)