Feature #28344
closed
Include HTTP Request2 for better HTTP handling
Added by Philipp Gampe almost 13 years ago.
Updated almost 11 years ago.
Description
t3lib_div::getUrl()
is know to be hudge and has a log of problems with edge cases. Especially it only implements a small subset of the HTTP protocol. This is a problem for extension which want to do some more advanced HTTP magic (for example linkvalidator).
A discussion on v4 list suggested to implement pear/HTTP_Request2 and use t3lib_div::getUrl
is a wrapper.
This is the first step which includes HTTP_Request2 and adds t3lib_utility_httpRequest
as wrapper which sets default options from TYPO3_CONF_VARS
.
Additionally some new TYPO3_CONF_VARS
have been added and the old curl options have been deprecated.
The next (and not included step) is to refactor t3lib_div::getUrl
.
Patch set 1 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
Patch set 2 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
- Status changed from New to Under Review
Patch set 3 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
Patch set 4 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
People still using PEAR? How about Symfony instead? Did anybody look at their HTTP bundle?
@Ingo could you please add a link to the Symfony HTTP bundle? I can't find it, but would like to take a look.
- Priority changed from Should have to Must have
Patch set 5 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
Patch set 6 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
Patch set 7 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
Patch set 8 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
Patch set 9 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437
- Status changed from Under Review to Resolved
- % Done changed from 80 to 100
Should t3lib_div::getUrl() use this new API now and maybe be deprecated?
Yes: see #28626, but we cannot deprecate it, because it is used so much.
- Status changed from Resolved to Closed
- Target version deleted (
4.6.0-beta1)
Also available in: Atom
PDF