Bug #90228: Typolink: Forcing absolute url scheme does not work - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #90228

closed

Typolink: Forcing absolute url scheme does not work

Added by Mario Lubenka almost 5 years ago. Updated about 3 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Frontend

Target version:

Candidate for patchlevel

Start date:

2020-01-28

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

easy

Is Regression:

Sprint Focus:

Description

According to documentation it is possible to force a url scheme (like https) as below:

page.1 = TEXT
page.1.typolink.parameter = 2
page.1.typolink.returnLast = url
page.1.typolink.forceAbsoluteUrl = 1
page.1.typolink.forceAbsoluteUrl.scheme = https

However the generated URL still only has the http protocol.

Looking into the PageUrlBuilder the corresponding line $url = $this->forceAbsoluteUrl($url, $conf); is only run for access-restricted pages.

Applying that to all urls by moving it down a line works for the scheme.
However the RegEx that is used seems to miss out on the / between base url and path.
So instead of "https://mypage.site/home" it generates "https://mypage.sitehome".

This RegEx seems to work better: #^(?:([a-z]+)(://)([^/]*)(/?))?(.*)$# (see RegEx in action)

I only tested this for TYPO3 9.5.13. Might affect 8.7 as well.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Mario Lubenka almost 5 years ago

Description updated (diff)

Actions

Copy link

Updated by Mario Lubenka almost 5 years ago

So... simply moving the "forceAbsoluteUrl" seems to break Frontend rendering. :(
And the new RegEx might not be necessary. It might be enough to add the missing / before the path.

Actions

Copy link

Updated by Gerrit Code Review almost 5 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/c/Packages/TYPO3.CMS/+/63252

Actions

Copy link

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/63252

Actions

Copy link

Updated by Thomas Oliver Moll over 4 years ago

Hi, is this patch complete?

As I read the bugreport PageUrlBuilder should also be modified not just AbstractTypolinkBuilder.php

I experience this problem in 9.5.17:

 stdWrap.typolink{
      returnLast = url
      forceAbsoluteUrl = 1
      forceAbsoluteUrl.scheme = https
      parameter.data = page:uid
      parameter.wrap=|,9818
      addQueryString = 1
      addQueryString.exclude = id,cHash,no_cache
      addQueryString.method = GET
      useCacheHash = 1
      no_cache = 0
  }

Or do I have to alter my typoscript?

Actions

Copy link

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/63252

Actions

Copy link

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/63252

Actions

Copy link

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/63252

Actions

Copy link

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/63252

Actions

Copy link

#10

Updated by Gerrit Code Review over 3 years ago

Patch set 1 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/69276

Actions

Copy link

#11