Skip to content

docs: add PSR-7 request body handling guide#3

Merged
TDannhauer merged 1 commit intoFRAMEWORK_6_0from
docs/psr7-request-body
Mar 12, 2026
Merged

docs: add PSR-7 request body handling guide#3
TDannhauer merged 1 commit intoFRAMEWORK_6_0from
docs/psr7-request-body

Conversation

@ralflang
Copy link
Copy Markdown
Member

@ralflang ralflang commented Mar 7, 2026

Documents best practices for PSR-7 request body stream handling, including non-seekable stream caveats, body parsing patterns, and integration with middleware like JsonBodyParser.

Add comprehensive documentation explaining the php://input non-seekable
stream issue and how to properly handle request bodies in PSR-7
applications.

Covers:
- Why php://input can only be read once
- The PSR-7 solution: getParsedBody()
- How to use JsonBodyParser middleware
- Best practices for body handling
- Rejected alternatives and rationale

This is a fundamental PSR-7 pattern that applies to all PHP
frameworks, not specific to Horde.
@TDannhauer TDannhauer merged commit 8f34335 into FRAMEWORK_6_0 Mar 12, 2026
0 of 6 checks passed
ralflang added a commit that referenced this pull request Mar 24, 2026
Release version 3.0.0-beta3

Merge pull request #3 from horde/docs/psr7-request-body
docs: document PSR-7 request body handling patterns
ralflang added a commit that referenced this pull request Mar 24, 2026
Release version 3.0.0-beta3

Merge pull request #3 from horde/docs/psr7-request-body
docs: document PSR-7 request body handling patterns
ralflang added a commit that referenced this pull request Mar 24, 2026
Release version 3.0.0-beta4

docs: Move TESTING.md to the doc folder
chore(release): bump version to 3.0.0-beta3
chore(dependencies): Drop dependency on horde/test no longer needed
feat(test): Separate unit test suite into live (real network) and unit (mock) tests.
Merge pull request #3 from horde/docs/psr7-request-body
docs: document PSR-7 request body handling patterns
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.

2 participants