Bug #37770
HTTP Request does not contain port information
Status: | Resolved | Start date: | 2012-06-05 | |
---|---|---|---|---|
Priority: | Must have | Due date: | ||
Assigned To: | Adrian Föder | % Done: | 100% |
|
Category: | Http | |||
Target version: | TYPO3 Flow Base Distribution - 1.1 beta 3 | |||
PHP Version: | Complexity: | |||
Has patch: | Yes | Affected Flow version: | Git master |
Description
Port information is not regarded when creating a new Request form ::create factory method
Related issues
Associated revisions
[BUGFIX] Non-standard ports are now considered by HttpRequest
Ports that are not 80 for HTTP or 443 for HTTPS were simply not
passed through to the Request when being created by the create()
method.
Also Uri will now set the port to 443 for HTTPS if no port is
given in the string handed to the constructor.
Change-Id: I52be3aa3d04cb9407c663a5df4501e6ed04c9995
Fixes: #37770
Releases: 1.1, 1.2
[BUGFIX] Non-standard ports are now considered by HttpRequest
Ports that are not 80 for HTTP or 443 for HTTPS were simply not
passed through to the Request when being created by the create()
method.
Also Uri will now set the port to 443 for HTTPS if no port is
given in the string handed to the constructor.
Change-Id: I52be3aa3d04cb9407c663a5df4501e6ed04c9995
Fixes: #37770
Releases: 1.1, 1.2
[BUGFIX] BaseUri contains default HTTPS port
By resolving #37770 now the BaseUri and thus the base tag when
using <f:base /> contains the HTTP port if it's not equal to 80.
This adjusts Request and Uri to omit the port if it is equal to
443 and the request scheme is https.
Change-Id: Ie62bce984e23b829e25c7ba9ca00e527e79958b1
Fixes: #38547
Related: #37770
Releases: 1.1, 1.2
[BUGFIX] BaseUri contains default HTTPS port
By resolving #37770 now the BaseUri and thus the base tag when
using <f:base /> contains the HTTP port if it's not equal to 80.
This adjusts Request and Uri to omit the port if it is equal to
443 and the request scheme is https.
Change-Id: Ie62bce984e23b829e25c7ba9ca00e527e79958b1
Fixes: #38547
Related: #37770
Releases: 1.1, 1.2
History
#1 Updated by Gerrit Code Review about 3 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#2 Updated by Gerrit Code Review about 3 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#3 Updated by Gerrit Code Review about 3 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#4 Updated by Gerrit Code Review about 3 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#5 Updated by Gerrit Code Review about 3 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#6 Updated by Gerrit Code Review about 3 years ago
Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#7 Updated by Gerrit Code Review about 3 years ago
Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#8 Updated by Gerrit Code Review about 3 years ago
Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#9 Updated by Gerrit Code Review about 3 years ago
Patch set 1 for branch FLOW3-1.1 has been pushed to the review server.
It is available at http://review.typo3.org/12192
#10 Updated by Gerrit Code Review about 3 years ago
Patch set 9 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11839
#11 Updated by Gerrit Code Review about 3 years ago
Patch set 2 for branch FLOW3-1.1 has been pushed to the review server.
It is available at http://review.typo3.org/12192
#12 Updated by Adrian Föder about 3 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 091a37f7ab8332fe69b79df5659bf1559235099a.
#13 Updated by Karsten Dambekalns about 3 years ago
- Target version changed from 1.1 RC1 to 1.1 beta 3