Fix for deadlock with large report (#508)

* add large xml sample * Avoid deadlock * Remove extra whitespaces
domainaware · May 22, 2024 · 7aa2e14 · 7aa2e14
1 parent 0b46c18
commit 7aa2e14
Show file tree

Hide file tree

Showing 2 changed files with 45,743 additions and 3 deletions.
diff --git a/parsedmarc/cli.py b/parsedmarc/cli.py
@@ -1194,15 +1194,15 @@ def process_reports(reports_):
         for proc in processes:
             proc.start()
 
+        for conn in connections:
+            results.append(conn.recv())
+
         for proc in processes:
             proc.join()
             if sys.stdout.isatty():
                 counter += 1
                 pbar.update(counter - pbar.n)
 
-        for conn in connections:
-            results.append(conn.recv())
-
     for result in results:
         if type(result[0]) is ParserError:
             logger.error("Failed to parse {0} - {1}".format(result[1],