Add sterr to output list (#3649)

Co-authored-by: Jan Klopper <[email protected]>
minvws · Oct 23, 2024 · 64461ad · 64461ad
1 parent 870b4fe
commit 64461ad
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/boefjes/boefjes/plugins/kat_dnssec/main.py b/boefjes/boefjes/plugins/kat_dnssec/main.py
@@ -17,4 +17,8 @@ def run(boefje_meta: dict):
     cmd = ["/usr/bin/drill", "-DT", domain]
     output = subprocess.run(cmd, capture_output=True)
 
-    return [({"openkat/dnssec-output"}, output.stdout)]
+    output.check_returncode()
+
+    results = [({"openkat/dnssec-output"}, output.stdout)]
+
+    return results
diff --git a/boefjes/boefjes/plugins/kat_nmap_tcp/main.py b/boefjes/boefjes/plugins/kat_nmap_tcp/main.py
@@ -18,4 +18,10 @@ def run(boefje_meta: dict):
         cmd.append("-6")
 
     cmd.extend(["-oX", "-", str(ip)])
-    return [({"openkat/nmap-output"}, subprocess.run(cmd, capture_output=True).stdout.decode())]
+    output = subprocess.run(cmd, capture_output=True)
+
+    output.check_returncode()
+
+    results = [({"openkat/nmap-output"}, output.stdout.decode())]
+
+    return results