Skip to content

FOUR-28073 Canceled case still progresses to Completed#8801

Merged
nolanpro merged 7 commits into
developfrom
bugfix/FOUR-28073
May 7, 2026
Merged

FOUR-28073 Canceled case still progresses to Completed#8801
nolanpro merged 7 commits into
developfrom
bugfix/FOUR-28073

Conversation

@gproly
Copy link
Copy Markdown
Contributor

@gproly gproly commented Apr 28, 2026

Description:
Race: cancel closed only in-memory tokens while parallel gateways could
create new ACTIVE tokens afterward; completing them overwrote CANCELED
with COMPLETED. Bulk-close stray tokens on cancel, block task completion
and new activations when status is CANCELED, skip COMPLETED persistence
if already canceled, guard script completion, and rethrow HttpResponseException
from BPMN actions so 422 responses are not swallowed as request errors.

Related tickets:
https://processmaker.atlassian.net/browse/FOUR-28073

ci:deploy

gproly added 3 commits April 28, 2026 14:40
Description:
Race: cancel closed only in-memory tokens while parallel gateways could
create new ACTIVE tokens afterward; completing them overwrote CANCELED
with COMPLETED. Bulk-close stray tokens on cancel, block task completion
and new activations when status is CANCELED, skip COMPLETED persistence
if already canceled, guard script completion, and rethrow HttpResponseException
from BPMN actions so 422 responses are not swallowed as request errors.

Related tickets:
https://processmaker.atlassian.net/browse/FOUR-28073
@Kookster310
Copy link
Copy Markdown
Contributor

QA server K8S was successfully deployed https://ci-778d7fdf17.engk8s.processmaker.net

@gproly gproly requested a review from caleeli April 29, 2026 20:30
Comment thread ProcessMaker/Jobs/BpmnAction.php Outdated
Comment thread ProcessMaker/Jobs/RunNayraScriptTask.php Outdated
Comment thread ProcessMaker/Jobs/RunNayraScriptTask.php Outdated
Comment thread ProcessMaker/Nayra/Managers/WorkflowManagerDefault.php Outdated
@gproly gproly requested a review from caleeli May 6, 2026 21:04
@processmaker-sonarqube
Copy link
Copy Markdown

@Kookster310
Copy link
Copy Markdown
Contributor

QA server K8S was successfully deployed https://ci-778d7fdf17.engk8s.processmaker.net

@nolanpro nolanpro merged commit 27c63c9 into develop May 7, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants