Bug #38547

BaseUri contains default HTTPS port

Added by Bastian Waidelich about 3 years ago. Updated about 3 years ago.

Status:Resolved Start date:2012-07-02
Priority:Should have Due date:
Assigned To:Bastian Waidelich % Done:

100%

Category:Http
Target version:-
PHP Version: Complexity:
Has patch:No Affected Flow version:Git 1.1

Description

By resolving #37770 now the BaseUri (thus the base tag when using <f:base />) contains the HTTP port if it's not 80.
It should also be omitted for 443 (HTTPS)


Related issues

related to TYPO3.Flow - Bug #37770: HTTP Request does not contain port information Resolved 2012-06-05

Associated revisions

Revision 58354078
Added by Bastian Waidelich about 3 years ago

[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

Revision fc7b8774
Added by Bastian Waidelich about 3 years ago

[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 Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12515

#2 Updated by Bastian Waidelich about 3 years ago

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

#3 Updated by Gerrit Code Review about 3 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch FLOW3-1.1 has been pushed to the review server.
It is available at http://review.typo3.org/12610

#4 Updated by Bastian Waidelich about 3 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF