This commit is contained in:
shamoon
2025-12-28 21:02:02 -08:00
parent 5fbc985b67
commit 2332b3f6ad

View File

@@ -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( def execute_password_removal_action(
action: WorkflowAction, action: WorkflowAction,
document: Document, document: Document,
@@ -284,7 +270,7 @@ def execute_password_removal_action(
""" """
Try to remove a password from a document using the configured list. Try to remove a password from a document using the configured list.
""" """
passwords = parse_passwords(action.passwords) passwords = action.passwords
if not passwords: if not passwords:
logger.warning( logger.warning(
"Password removal action %s has no passwords configured", "Password removal action %s has no passwords configured",
@@ -293,6 +279,12 @@ def execute_password_removal_action(
) )
return return
passwords = [
password.strip()
for password in re.split(r"[,\n]", passwords)
if password.strip()
]
# import here to avoid circular dependency # import here to avoid circular dependency
from documents.bulk_edit import remove_password from documents.bulk_edit import remove_password