Skip to content

fix(engine): harden workflow joins, cancellation and logging - #87

Merged
Zhoujie628 merged 1 commit into
project-openan:mainfrom
hanyu0504:fix/engine-workflow-hardening
Sep 23, 2026
Merged

Zhoujie628 merged 1 commit into
project-openan:mainfrom
hanyu0504:fix/engine-workflow-hardening

Conversation

@hanyu0504

Copy link
Copy Markdown
Contributor

Summary

  • Wait for all active ancestors before executing a DAG join; propagate cancellation to pending task and route callbacks.
  • Preserve structured subtask input, validate poll intervals, and redact additional credential keys.
  • Remove business payloads from operational logs and refresh English/Chinese integration version references.

The dev workspace commit also removes the duplicate log4j2.properties charset setting. That duplicate is already absent on upstream main, so the PR contains no redundant change to that file.

Verification

  • mvn -B -pl workflow-engine -am test (238 tests passed on main)
  • mvn -B -pl samples -am '-Dtest=SpringSpnDemoE2ETest' '-Dsurefire.failIfNoSpecifiedTests=false' test (4 Direct E2E tests passed)
  • On dev, Direct + Order E2E: 8 tests passed.

Note: the baseline full reactor still has a separate samples SSE terminal-cancel logging assertion failure; not changed in this PR.

Signed-off-by: hanyu0504 <zhouyulong@gdhurray.com>
@hanyu0504
hanyu0504 requested a review from a team as a code owner September 23, 2026 07:30
@Zhoujie628
Zhoujie628 merged commit d72c86b into project-openan:main Sep 23, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants