Feature #28344: Include HTTP Request2 for better HTTP handling - TYPO3 Core - TYPO3 Forge

Actions

Feature #28344

closed

Include HTTP Request2 for better HTTP handling

Added by Philipp Gampe over 12 years ago. Updated almost 11 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

Miscellaneous

Target version:

-

Start date:

2011-07-20

Due date:

2011-07-25

% Done:

100%

Estimated time:

4.00 h

PHP Version:

5.2

Tags:

Complexity:

Sprint Focus:

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.

Related issues 8 (0 open — 8 closed)

Related to TYPO3 Core - Feature #27988: Override / add curl options to t3lib_div::getUrl

Rejected

2011-07-08

Related to TYPO3 Core - Bug #20770: [t3lib_div::getURL][curl] CURLOPT_HTTPAUTH missing

Closed

2009-07-19

Related to TYPO3 Core - Bug #25321: getUrl crashes when trying to retrieve a big file

Rejected

2011-03-14

Related to TYPO3 Core - Feature #28839: Add USERAGENT to cURL request in t3lib_div#getURL()

Closed

2011-08-08

Related to TYPO3 Core - Bug #45834: Detection of curlProxyServer settings buggy on upgrade to 6.0

Closed

2013-02-25

Blocks TYPO3 Core - Task #28614: Add TYPO3 exceptions to \TYPO3\CMS\Core\Http\HttpRequest

Closed

Sebastian Michaelsen

2011-07-31

Blocks TYPO3 Core - Feature #28615: Add logging observer to t3lib_http_Request

Rejected

2011-07-31

Blocks TYPO3 Core - Feature #28626: Refactor t3lib_div::getUrl()

Closed

2011-08-01

Actions

#1

Updated by Mr. Hudson over 12 years ago

Patch set 1 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#2

Updated by Mr. Hudson over 12 years ago

Patch set 2 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#3

Updated by Tolleiv Nietsch over 12 years ago

Status changed from New to Under Review

Actions

#4

Updated by Mr. Hudson over 12 years ago

Patch set 3 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#5

Updated by Mr. Hudson over 12 years ago

Patch set 4 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#6

Updated by Ingo Renner over 12 years ago

People still using PEAR? How about Symfony instead? Did anybody look at their HTTP bundle?

Actions

#7

Updated by Philipp Gampe over 12 years ago

@Ingo could you please add a link to the Symfony HTTP bundle? I can't find it, but would like to take a look.

Actions

#8

Updated by Xavier Perseguers over 12 years ago

Priority changed from Should have to Must have

Actions

#9

Updated by Mr. Hudson over 12 years ago

Patch set 5 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#10

Updated by Mr. Hudson over 12 years ago

Patch set 6 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#11

Updated by Mr. Hudson over 12 years ago

Patch set 7 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#12

Updated by Mr. Hudson over 12 years ago

Patch set 8 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#13

Updated by Mr. Hudson over 12 years ago

Patch set 9 of change I430685159cc966e13f1e833ca19ee8f0c75e400a has been pushed to the review server.
It is available at http://review.typo3.org/3437

Actions

#14

Updated by Philipp Gampe over 12 years ago

Status changed from Under Review to Resolved
% Done changed from 80 to 100

Applied in changeset addec7b35713a9cbd5b2fa170ccba0d1a9e113b2.

Actions

#15

Updated by Christian Kuhn over 12 years ago

Should t3lib_div::getUrl() use this new API now and maybe be deprecated?

Actions

#16

Updated by Philipp Gampe over 12 years ago

Yes: see #28626, but we cannot deprecate it, because it is used so much.

Actions

#17

Updated by Xavier Perseguers about 12 years ago

Status changed from Resolved to Closed

Actions

#18

Updated by Ernesto Baschny almost 11 years ago

Target version deleted (~~4.6.0-beta1~~)

Actions

Also available in: Atom PDF