From 2332b3f6ad7ce0c178b6a0d9e2600fd76b421032 Mon Sep 17 00:00:00 2001
From: shamoon <4887959+shamoon@users.noreply.github.com>
Date: Sun, 28 Dec 2025 21:02:02 -0800
Subject: [PATCH] and this

---
 src/documents/workflows/actions.py | 22 +++++++---------------
 1 file changed, 7 insertions(+), 15 deletions(-)

diff --git a/src/documents/workflows/actions.py b/src/documents/workflows/actions.py
index 37853ba8c..c18a5a6a2 100644
--- a/src/documents/workflows/actions.py
+++ b/src/documents/workflows/actions.py
@@ -262,20 +262,6 @@ def execute_webhook_action(
         )
 
 
-def parse_passwords(raw_passwords: str | None) -> list[str]:
-    """
-    Convert a comma/newline separated string of passwords into a clean list.
-    """
-    if not raw_passwords:
-        return []
-
-    return [
-        password.strip()
-        for password in re.split(r"[,\n]", raw_passwords)
-        if password.strip()
-    ]
-
-
 def execute_password_removal_action(
     action: WorkflowAction,
     document: Document,
@@ -284,7 +270,7 @@ def execute_password_removal_action(
     """
     Try to remove a password from a document using the configured list.
     """
-    passwords = parse_passwords(action.passwords)
+    passwords = action.passwords
     if not passwords:
         logger.warning(
             "Password removal action %s has no passwords configured",
@@ -293,6 +279,12 @@ def execute_password_removal_action(
         )
         return
 
+    passwords = [
+        password.strip()
+        for password in re.split(r"[,\n]", passwords)
+        if password.strip()
+    ]
+
     # import here to avoid circular dependency
     from documents.bulk_edit import remove_password