DOCS: Add log and nosec in checked subprocess calls (#6001)

Co-authored-by: pyansys-ci-bot <[email protected]>
Co-authored-by: Maxime Rey <[email protected]>

Author: 3 people

doc/changelog.d/6001.documentation.md

@@ -0,0 +1 @@
+Add log and nosec in checked subprocess calls

doc/source/Resources/pyaedt_installer_from_aedt.py

@@ -29,9 +29,9 @@
 import sys
 
 try:
-    import subprocess
+    import subprocess  # nosec
 except ImportError:
-    import subprocessdotnet as subprocess
+    import subprocessdotnet as subprocess  # nosec
 
 is_iron_python = platform.python_implementation().lower() == "ironpython"
 is_linux = os.name == "posix"
@@ -99,7 +99,7 @@ def run_pyinstaller_from_c_python(oDesktop):
         command.extend([r'--wheel={}'.format(wheelpyaedt)])
 
     oDesktop.AddMessage("", "", 0, "Installing PyAEDT.")
-    return_code = subprocess.call(command)
+    return_code = subprocess.call(command)  # nosec
 
     err_msg = "There was an error while installing PyAEDT."
     if is_linux:
@@ -153,7 +153,7 @@ def run_pyinstaller_from_c_python(oDesktop):
 
     command = r'"{}" "{}"'.format(python_exe, python_script)
     oDesktop.AddMessage("", "", 0, "Configuring PyAEDT panels in automation tab.")
-    ret_code = subprocess.call([python_exe, python_script])
+    ret_code = subprocess.call([python_exe, python_script])  # nosec
     if ret_code != 0:
         oDesktop.AddMessage("", "", 2, "Error occurred configuring the PyAEDT panels.")
         return
@@ -249,9 +249,9 @@ def install_pyaedt():
     if not venv_dir.exists():
         print("Creating the virtual environment in {}".format(venv_dir))
         if args.version <= "231":
-            subprocess.run([sys.executable, "-m", "venv", str(venv_dir), "--system-site-packages"], check=True)
+            subprocess.run([sys.executable, "-m", "venv", str(venv_dir), "--system-site-packages"], check=True)  # nosec
         else:
-            subprocess.run([sys.executable, "-m", "venv", str(venv_dir)], check=True)
+            subprocess.run([sys.executable, "-m", "venv", str(venv_dir)], check=True)  # nosec
 
         if args.wheel and Path(args.wheel).exists():
             print("Installing PyAEDT using provided wheels argument")

doc/source/Resources/toolkit_installer_from_aedt.py

@@ -24,7 +24,7 @@
 
 import os
 import sys
-import subprocessdotnet as subprocess
+import subprocessdotnet as subprocess  # nosec
 
 # This script installs PyAEDT tabs (PyAEDT Console, Jupyter, Run Script and Extension Manager)
 # using a specific Python interpreter.
@@ -58,7 +58,7 @@ def run_pyinstaller_from_c_python(oDesktop, pyaedt_interpreter):
 
     command = [pyaedt_interpreter, python_script]
     oDesktop.AddMessage("", "", 0, "Configuring PyAEDT panels in automation tab.")
-    process = subprocess.Popen(command)
+    process = subprocess.Popen(command)  # nosec
     process.wait()
 
     # Refresh UI
@@ -80,9 +80,11 @@ def run_pyinstaller_from_c_python(oDesktop, pyaedt_interpreter):
 if __name__ == "__main__":
 
     python_interpreter = os.getenv(pyaedt_enviroment_variable)
-    if python_interpreter and not os.path.exists(python_interpreter):
-        oDesktop.AddMessage("", "", 2, "Python environment does not exist.")
-        sys.exit()
+    if python_interpreter:
+        oDesktop.AddMessage("", "", 0, "Using Python environment defined with the environment variable PYAEDT_INTERPRETER.")
+        if os.path.exists(python_interpreter):
+            oDesktop.AddMessage("", "", 2, "Python environment does not exist.")
+            sys.exit()
 
     # Check if interpreter path is defined.
     # Retrieve the script arguments