Bug #85873
closedTypolink not contains port on absolute URIs only, if sys_domain includes port
0%
Description
Generation of an absolute TypoLink with a non-standard port fails (On TYPO3 7.6 it works as excepted).
In 7.6 there was an additional check (ContentObjectRenderer
)
$tsfe->config['config']['typolinkCheckRootline']
that not on our Installation not matched and so target domain is set to current domain here:
// If no domain records are defined, use current domain:
$currentUrlScheme = parse_url($this->getEnvironmentVariable('TYPO3_REQUEST_URL'), PHP_URL_SCHEME);
if ($targetDomain === '' && ($conf['forceAbsoluteUrl'] || $absoluteUrlScheme !== $currentUrlScheme)) {
$targetDomain = $currentDomain;
}
As result: $currentDomain
and $targetDomain
are equals and contains the port.
In 8.7 and 9.x the typolinkCheckRootline was removed in PageLinkBuilder
, so currentDomain set to the sys_domain-Record, which should not contains the port. This means, that $currentDomain
!= $targetDomain
, so PageLinkBuilder build the url, but missed the port
Updated by Gerrit Code Review over 6 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57929
Updated by Gerrit Code Review over 6 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57929
Updated by Gerrit Code Review over 6 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57929
Updated by Gerrit Code Review about 6 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57929
Updated by Gerrit Code Review about 6 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57929
Updated by Gerrit Code Review about 6 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57929
Updated by Benni Mack about 6 years ago
- Status changed from Under Review to Rejected