From 9c627307b862647c31d6b0fd544360fdcd71a68a Mon Sep 17 00:00:00 2001
From: kshitij <82637225+kshitijk4poor@users.noreply.github.com>
Date: Fri, 17 Jan 2025 12:04:54 +0530
Subject: [PATCH] lint

---
 artemis/modules/moodle_scanner.py | 38 +++++++------------------------
 1 file changed, 8 insertions(+), 30 deletions(-)

diff --git a/artemis/modules/moodle_scanner.py b/artemis/modules/moodle_scanner.py
index 4e4a9beed..61bd14192 100644
--- a/artemis/modules/moodle_scanner.py
+++ b/artemis/modules/moodle_scanner.py
@@ -1,19 +1,15 @@
 #!/usr/bin/env python3
 import dataclasses
-import json
-import os
 import subprocess
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, List
 
 from karton.core import Task
 
 from artemis import load_risk_class
 from artemis.binds import Service, TaskStatus, TaskType
-from artemis.config import Config
 from artemis.module_base import ArtemisBase
 from artemis.task_utils import get_target_url
 
-
 @dataclasses.dataclass
 class MoodleMessage:
     category: str
@@ -23,7 +19,6 @@ class MoodleMessage:
     def message(self) -> str:
         return f"{self.category}: {', '.join(self.problems)}"
 
-
 def process_moodle_json(result: Dict[str, Any]) -> List[MoodleMessage]:
     messages: Dict[str, MoodleMessage] = {}
 
@@ -60,7 +55,6 @@ def process_moodle_json(result: Dict[str, Any]) -> List[MoodleMessage]:
 
     return list(messages.values())
 
-
 @load_risk_class.load_risk_class(load_risk_class.LoadRiskClass.MEDIUM)
 class MoodleScanner(ArtemisBase):
     """
@@ -82,20 +76,13 @@ def run(self, current_task: Task) -> None:
         try:
             # Run moodlescan with error output captured
             process = subprocess.run(
-                [
-                    "python3",
-                    "moodlescan.py",
-                    "-u",
-                    base_url,
-                    "-r",
-                    "-k"
-                ],
+                ["python3", "moodlescan.py", "-u", base_url, "-r", "-k"],
                 cwd="/moodle_scanner",
                 capture_output=True,
                 text=True,
-                check=True
+                check=True,
             )
-            
+
             self.log.info(f"Moodlescan stdout: {process.stdout}")
             if process.stderr:
                 self.log.warning(f"Moodlescan stderr: {process.stderr}")
@@ -110,13 +97,10 @@ def run(self, current_task: Task) -> None:
                 if "Error: Can't connect" in line:
                     self.log.info(f"Connection error: {line}")
                     self.db.save_task_result(
-                        task=current_task,
-                        status=TaskStatus.OK,
-                        status_reason=line,
-                        data={"raw_output": process.stdout}
+                        task=current_task, status=TaskStatus.OK, status_reason=line, data={"raw_output": process.stdout}
                     )
                     return
-                    
+
                 if "server" in line.lower() and ":" in line:
                     server_info = line.split(":", 1)[1].strip()
                 elif "version" in line.lower() and not line.startswith("."):
@@ -132,7 +116,7 @@ def run(self, current_task: Task) -> None:
                 "server": server_info,
                 "version": version_info,
                 "vulnerabilities": vulnerabilities,
-                "raw_output": process.stdout
+                "raw_output": process.stdout,
             }
 
             # Determine if anything interesting was found
@@ -146,12 +130,7 @@ def run(self, current_task: Task) -> None:
                 status = TaskStatus.OK
                 status_reason = "Version not found" if version_info == "Version not found" else None
 
-            self.db.save_task_result(
-                task=current_task,
-                status=status,
-                status_reason=status_reason,
-                data=result
-            )
+            self.db.save_task_result(task=current_task, status=status, status_reason=status_reason, data=result)
 
         except subprocess.CalledProcessError as e:
             self.log.error(f"Failed to run moodlescan for {base_url}")
@@ -166,6 +145,5 @@ def run(self, current_task: Task) -> None:
             )
             return
 
-
 if __name__ == "__main__":
     MoodleScanner().loop()