diff --git a/src/documents/tests/test_workflows.py b/src/documents/tests/test_workflows.py index ad9768a17..2e31608bb 100644 --- a/src/documents/tests/test_workflows.py +++ b/src/documents/tests/test_workflows.py @@ -4164,7 +4164,7 @@ class TestWorkflows( ) action = WorkflowAction.objects.create( type=WorkflowAction.WorkflowActionType.PASSWORD_REMOVAL, - passwords="wrong, right\n extra ", + passwords=["wrong", "right", "extra"], ) workflow = Workflow.objects.create(name="Password workflow") workflow.triggers.add(trigger) @@ -4218,7 +4218,7 @@ class TestWorkflows( ) action = WorkflowAction.objects.create( type=WorkflowAction.WorkflowActionType.PASSWORD_REMOVAL, - passwords=" \n , ", + passwords=[" ", " "], ) workflow = Workflow.objects.create(name="Password workflow missing passwords") workflow.triggers.add(trigger) @@ -4276,7 +4276,7 @@ class TestWorkflows( """ action = WorkflowAction.objects.create( type=WorkflowAction.WorkflowActionType.PASSWORD_REMOVAL, - passwords="first, second", + passwords=["first", "second"], ) temp_dir = Path(tempfile.mkdtemp()) diff --git a/src/documents/workflows/actions.py b/src/documents/workflows/actions.py index 9089c6dd8..b6adc35b7 100644 --- a/src/documents/workflows/actions.py +++ b/src/documents/workflows/actions.py @@ -1,5 +1,4 @@ import logging -import re import uuid from pathlib import Path @@ -290,11 +289,7 @@ def execute_password_removal_action( ) return - passwords = [ - password.strip() - for password in re.split(r"[,\n]", passwords) - if password.strip() - ] + passwords = [p.strip() for p in passwords if p.strip()] if isinstance(document, ConsumableDocument): # hook the consumption-finished signal to attempt password removal later