remove process timeout from reachability analysis (#141)

* remove process timeout from reachability analysis

* Fixing version for build

---------

Co-authored-by: Douglas Coburn <[email protected]>

Author: mtorp

CHANGELOG.md

@@ -0,0 +1,5 @@
+# Changelog
+
+## 2.2.56
+
+- Removed process timeout from reachability analysis subprocess. Timeouts are now only passed to the Coana CLI via the `--analysis-timeout` flag.

README.md

@@ -527,3 +527,4 @@ pre-commit install
 ```
 
 > **Note**: This manual setup is an alternative to the streamlined Make targets described above. For most development workflows, using `make first-time-setup` or `make first-time-local-setup` is recommended.
+

pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "socketsecurity"
-version = "2.2.55"
+version = "2.2.56"
 requires-python = ">= 3.10"
 license = {"file" = "LICENSE"}
 dependencies = [

socketsecurity/__init__.py

@@ -1,3 +1,3 @@
 __author__ = 'socket.dev'
-__version__ = '2.2.55'
+__version__ = '2.2.56'
 USER_AGENT = f'SocketPythonCLI/{__version__}'

socketsecurity/core/tools/reachability.py

@@ -220,7 +220,6 @@ def run_reachability_analysis(
                 cwd=target_directory,
                 stdout=sys.stderr,  # Send stdout to stderr so user sees it
                 stderr=sys.stderr,  # Send stderr to stderr
-                timeout=timeout + 60 if timeout else None  # Add buffer to subprocess timeout
             )
 
             if result.returncode != 0:
@@ -240,9 +239,6 @@ def run_reachability_analysis(
                 "tar_hash_used": tar_hash
             }
 
-        except subprocess.TimeoutExpired:
-            log.error(f"Reachability analysis timed out after {timeout} seconds")
-            raise Exception(f"Reachability analysis timed out after {timeout} seconds")
         except Exception as e:
             log.error(f"Failed to run reachability analysis: {str(e)}")
             raise Exception(f"Failed to run reachability analysis: {str(e)}")