Skip to content

#123 HTTP API: read the query string without body, and queue from the query string - #124

Merged
njoubert-cleverage merged 2 commits into
mainfrom
123
Oct 8, 2026
Merged

njoubert-cleverage merged 2 commits into
mainfrom
123

Conversation

@njoubert-cleverage

Copy link
Copy Markdown
Member

Description

Fixes #123.

HttpProcessExecuteValueResolver:

  • the query string is also read when the request has no body (no form data, no JSON content): it was refused (422 Process code is required.)
  • queue is read from the query string when it is not in the body (it was only read from the body: ?queue=0 was ignored and the process queued)

The form data still takes precedence over the query string; a JSON body is handled as before. docs/reference/06-http_api.md updated.

Tests:

  • HttpProcessExecuteValueResolverTest: query string without body, queue from the query string, queue of the body taking precedence
  • HttpProcessExecuteTest::testQueryStringWithoutBody: synchronous execution with ?code=...&queue=0 and no body

Checked:

  • bundle (Symfony 8.1, PHP 8.5): 287 tests OK; without the fix, the new resolver tests fail. PHPStan, PHP-CS-Fixer, Rector OK
  • process-bundle-demo (Symfony 7.4): ?code=demo.debug&queue=0 without body → 200 "Process has been proceed well." (was 422); -F code=demo.debug with ?queue=0 → executed (was queued); default still queued

Requirements

  • Documentation updates
    • Reference
    • Changelog
  • Unit tests

Breaking changes

None: these requests were refused, or queue ignored.

🤖 Generated with Claude Code

… query string

A request with the parameters in the query string only was refused (422 "Process code is required"), and ?queue=0
was ignored (the process was queued). The form data takes precedence over the query string.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@njoubert-cleverage njoubert-cleverage added the enhancement New feature or request label Oct 8, 2026
@njoubert-cleverage
njoubert-cleverage merged commit 8364e22 into main Oct 8, 2026
18 checks passed
@njoubert-cleverage
njoubert-cleverage deleted the 123 branch October 8, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP API: query string ignored without body, queue not read from the query string

1 participant