Bug #93237
closed
TYPO3 should not extend PSR-17 interfaces for Extbase usage
100%
Description
..but extbase should rather use PSR-17 as is.
The interface TYPO3\CMS\Core\Http\ResponseFactory – introduced by https://review.typo3.org/c/Packages/TYPO3.CMS/+/66553 – contradicts the ideas of interchangable PSR interfaces and therefore we should strive for native PSR-17 usage, instead of wrapping the interfaces, instead.
Also, concrete PSR-7 implementation should be kept @internal, and only the instances of the interfaces (as aliased into the DI container), should be "public" api.
The Extbase ActionController is free to provide helper methods around the PSR ResponseFactory, but should use PSR-17 directly.
Updated by Gerrit Code Review almost 4 years ago
- Status changed from New to Under Review
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/67353
Updated by Gerrit Code Review almost 4 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/67353
Updated by Benjamin Franzke almost 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 71884c8fffcb085e1c669cd93ab656269f35ac3f.
Updated by