cleaned up license_details hack

Author: flowstate

README.md

@@ -10,6 +10,7 @@ socketcli [-h] [--api-token API_TOKEN] [--repo REPO] [--integration {api,github,
           [--target-path TARGET_PATH] [--sbom-file SBOM_FILE] [--files FILES] [--default-branch] [--pending-head] 
           [--generate-license] [--enable-debug] [--enable-json] [--enable-sarif] [--disable-overview] [--disable-security-issue] 
           [--allow-unverified] [--ignore-commit-files] [--disable-blocking] [--scm SCM] [--timeout TIMEOUT]
+          [--exclude-license-details]
 ````
 
 If you don't want to provide the Socket API Token every time then you can use the environment variable `SOCKET_SECURITY_API_KEY`
@@ -58,6 +59,7 @@ If you don't want to provide the Socket API Token every time then you can use th
 | --enable-json        | False    | False   | Output in JSON format                                         |
 | --enable-sarif       | False    | False   | Enable SARIF output of results instead of table or JSON format|
 | --disable-overview   | False    | False   | Disable overview output                                       |
+| --exclude-license-details | False    | False   | Exclude license details from the diff report (boosts performance for large repos) |
 
 #### Security Configuration
 | Parameter               | Required | Default | Description                    |

socketsecurity/__init__.py

@@ -1,2 +1,2 @@
 __author__ = 'socket.dev'
-__version__ = '2.0.6'
+__version__ = '2.0.7'

socketsecurity/config.py

@@ -33,6 +33,7 @@ class CliConfig:
     integration_org_slug: Optional[str] = None
     pending_head: bool = False
     timeout: Optional[int] = 1200
+    exclude_license_details: bool = False
     @classmethod
     def from_args(cls, args_list: Optional[List[str]] = None) -> 'CliConfig':
         parser = create_argument_parser()
@@ -71,6 +72,7 @@ def from_args(cls, args_list: Optional[List[str]] = None) -> 'CliConfig':
             'integration_type': args.integration,
             'pending_head': args.pending_head,
             'timeout': args.timeout,
+            'exclude_license_details': args.exclude_license_details,
         }
 
         if args.owner:
@@ -283,6 +285,12 @@ def create_argument_parser() -> argparse.ArgumentParser:
         action="store_true",
         help=argparse.SUPPRESS
     )
+    output_group.add_argument(
+        "--exclude-license-details",
+        dest="exclude_license_details",
+        action="store_true",
+        help="Exclude license details from the diff report (boosts performance for large repos)"
+    )
 
     # Security Configuration
     security_group = parser.add_argument_group('Security Configuration')

socketsecurity/core/__init__.py

@@ -462,7 +462,6 @@ def create_new_diff(
             has_head_scan = False
 
         # Create new scan
-        params.include_license_details = False
         new_scan_start = time.time()
         new_full_scan = self.create_full_scan(files_for_sending, params, has_head_scan)
         new_scan_end = time.time()
@@ -479,7 +478,12 @@ def create_new_diff(
 
         base_socket = "https://socket.dev/dashboard/org"
         diff.id = new_full_scan.id
-        diff.report_url = f"{base_socket}/{self.config.org_slug}/sbom/{diff.id}?include_license_details=false"
+
+        report_url = f"{base_socket}/{self.config.org_slug}/sbom/{diff.id}"
+        if not params.include_license_details:
+            report_url += "?include_license_details=false"
+        diff.report_url = report_url
+        
         if head_full_scan_id is not None:
             diff.diff_url = f"{base_socket}/{self.config.org_slug}/diff/{diff.id}/{head_full_scan_id}"
         else:

socketsecurity/socketcli.py

@@ -160,6 +160,8 @@ def main_code():
         set_as_pending_head=True
     )
 
+    params.include_license_details = not config.exclude_license_details
+
     # Initialize diff
     diff = Diff()
     diff.id = "NO_DIFF_RAN"