Bug #80220

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

Added by Nicole Cordes over 1 year ago. Updated about 2 months ago.

Status:
Closed
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 over 1 year 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 over 1 year 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 over 1 year 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 over 1 year 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 over 1 year ago

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

#5 Updated by Benni Mack about 2 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF