Resolve runtime payloads in service-operation workflow commands - #144
Conversation
|
Full local Feature suite: 2,062 tests and 41,663 assertions passed (three skips, six PHPUnit deprecations). The focused pair passed 72 tests/651 assertions. The new three-case regression failed on the target before the one-line production fix. Public CI is running. Review: this reuses the existing authenticated namespace claim/integrity path; it does not weaken size, lease, service authorization, or cross-namespace rules. Next: include the patch release identity and publish after checks pass. |
|
Merged source is a1bcc67, tree-identical to the qualified PR head. Immutable tag 2.3.3 starts the repository-owned publication run: https://github.com/durable-workflow/server/actions/runs/34216699544. Watching this exact run; #143 stays open until image/chart publication is verified. |
Addresses #143; paired with durable-workflow/sdk-php#63.
Resolve workflow-command request_payload through the existing namespace-authenticated runtime reference middleware. No codec, provider adapter, request-limit change, or admission bypass.
All three regressions fail on the original source and pass with the fix. Focused tests: 72 / 651 assertions. Full Feature suite: 2,062 tests / 41,663 assertions, three existing skips and six PHPUnit deprecations. Pint and all public CI passed, including rolling upgrade, replay/query HTTP, bounded-growth smoke and chart installation.
Prepares Server 2.3.3 and Helm 0.1.82. Qualification and transport-boundary review are in the comments. Issue closure follows publication.