Bug #80220

http_code not returned for HEAD requests in \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl

Added by Nicole Cordes 12 months ago. Updated 8 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
Category:
-
Target version:
Start date:
2017-03-10
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

If you switch to a HEAD request in \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl and provide a report array to get more information, the header status will not be return. This happens because the content from that request (obviously) is emtpy and considerer as an error.

Associated revisions

Revision 88fd7dfb (diff)
Added by Nicole Cordes 8 months ago

[BUGFIX] Return http_code for HEAD requests in GeneralUtility::getUrl

As there is no content available for a HEAD request
GeneralUtility::getUrl currently returns an error message for HEAD
requests instead of proper http_code information. This patch ensures
all information are available.

Resolves: #80220
Releases: 7.6
Change-Id: I64108a16846a294f0bbee07289afcb79474833ea
Reviewed-on: https://review.typo3.org/51994
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>

History

#1 Updated by Gerrit Code Review 12 months ago

  • Status changed from New to Under Review

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51994

#2 Updated by Gerrit Code Review 11 months ago

Patch set 2 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51994

#3 Updated by Gerrit Code Review 8 months ago

Patch set 3 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51994

#4 Updated by Nicole Cordes 8 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF