[loks0n]

2.1.5 ships Swoole v6.2.4, which treats a client-supplied Swoole-Upload-File multipart part header as an upload and unlinks the path it names when the request ends. v6.2.5 ignores the header (swoole/swoole-src#6309).

• Swoole v6.2.4 → v6.2.5 for PHP 8.3, 8.4 and 8.5.
• tests-swoole.php runs in the structure job against every built image: it posts the header naming an existing file and fails if the file is gone. Same check as fix: hold swoole at 6.2.3 and test upload marker handling appwrite/docker-base#135, without the handler clearing tmpfiles, so it tests Swoole alone.

Seen red and green locally on PHP 8.5:

• appwrite/utopia-base:php-8.5-2.1.5 (Swoole 6.2.4): status 200, target deleted, exit 1
• this branch (Swoole 6.2.5): status 200, target kept, exit 0

After merge: release 2.1.6.

🤖 Generated with Claude Code

[appwrite]

🟢 Tier S · Ready to merge

The version updates and the CI regression check are consistent with the described fix, with no actionable issues identified.

Updates Swoole to v6.2.5 for PHP 8.3, 8.4, and 8.5. Adds a CI check that sends a multipart request with a client-supplied Swoole-Upload-File part header and verifies the named file remains present.

Verdict	New comments	Fixed	Still open
✅ Approved	0	0	0

📂 Walkthrough · 5

File	Change
.github/workflows/ci.yml	Runs the Swoole upload-marker regression check against each built image in the structure-test job.
tests-swoole.php	Starts a local Swoole HTTP server, sends the crafted multipart request, and fails if the target file is deleted.
versions/8.3.env	Bumps the Swoole source version to v6.2.5.
versions/8.4.env	Bumps the Swoole source version to v6.2.5.
versions/8.5.env	Bumps the Swoole source version to v6.2.5.

---

_{Reviewed 6e785a8 · Details · Comment @appwrite review to re-run, or mention @appwrite with a question.}

[appwrite]

🟢 Tier S · Looks good to merge. Summary