Option to soften lanes (#2507)

* add option to not use lane for task assignment w/ burnettk

* lint

* add comments

---------

Co-authored-by: jasquat <[email protected]>

Author: jasquat

spiffworkflow-backend/src/spiffworkflow_backend/config/default.py

@@ -252,6 +252,10 @@ def config_from_env(variable_name: str, *, default: str | bool | int | None = No
 # check all tasks listed as child tasks are saved to the database
 config_from_env("SPIFFWORKFLOW_BACKEND_DEBUG_TASK_CONSISTENCY", default=False)
 
+# When set to False, this will use the initiator for all task assignments.
+# This is useful when using arena with api keys only and doing task assignment in a differnt system.
+config_from_env("SPIFFWORKFLOW_BACKEND_USE_LANES_FOR_TASK_ASSIGNMENT", default=True)
+
 ### for documentation only
 # we load the CustomBpmnScriptEngine at import time, where we do not have access to current_app,
 # so instead of using config, we use os.environ directly over there.

spiffworkflow-backend/src/spiffworkflow_backend/services/process_instance_processor.py

@@ -957,8 +957,10 @@ def raise_if_no_potential_owners(self, potential_owners: list[PotentialOwner], m
     def get_potential_owners_from_task(self, task: SpiffTask) -> PotentialOwnerIdList:
         task_spec = task.task_spec
         task_lane = "process_initiator"
-        if task_spec.lane is not None and task_spec.lane != "":
-            task_lane = task_spec.lane
+
+        if current_app.config.get("SPIFFWORKFLOW_BACKEND_USE_LANES_FOR_TASK_ASSIGNMENT") is not False:
+            if task_spec.lane is not None and task_spec.lane != "":
+                task_lane = task_spec.lane
 
         potential_owners: list[PotentialOwner] = []
         lane_assignment_id = None