Bug #84173
closed
TYPO3 8.7 GuzzleHttp \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl() returns content_type as array instead of string
100%
Description
There is a breaking change in the report format of \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl() introduced by the change form cURL to GuzzleHttp in #70056.
The old cURL based implementation returned the content_type in the $report as a string fetched via curl_getinfo(), but the new code uses $response->getHeader('Content-Type') which returns an array of strings.
In order to not break backward compatibility the code should use $response->getHeaderLine('Content-Type') instead.
Updated by Felix Buenemann about 6 years ago
- Related to Feature #70056: Use guzzle added
Updated by Felix Buenemann about 6 years ago
- Related to Bug #84167: Negation Bug in TYPO3 8.7 GuzzleHttp \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl() added
Updated by Felix Buenemann about 6 years ago
This is also related to #84167, since due to that bug the content_type was not returned in most cases (only for 300-400 status responses).
Updated by Gerrit Code Review about 6 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56043
Updated by Gerrit Code Review about 6 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56043
Updated by Gerrit Code Review about 6 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56057
Updated by Felix Buenemann about 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset bed7b8005be42ac55f32a9f323898bf37c3b8a47.
Updated by