Bug #97303
closedContentObjectRenderer http_makelinks replace https URLs with http URLs
100%
Description
having a Fluid-Template with
<f:format.html>https://foo.bar.de/</f:format.html>
renders
<p><a href="http://foo.bar.de/" target="_blank" rel="noreferrer">foo.bar.de</a></p>
same error when using lib.parseFunc instead of lib.parseFunc_RTE
<f:format.html parseFuncTSPath="lib.parseFunc">https://foo.bar.de/</f:format.html>
renders:
<a href="http://foo.bar.de/" target="_blank" rel="noreferrer">foo.bar.de</a>
i think the schema should be readdet when calling typolink in http_makelinks
index debc7b9e74..48c3f1ec02 100644
--- a/typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php
+++ b/typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php
@@ -3743,7 +3743,7 @@ class ContentObjectRenderer implements LoggerAwareInterface
}
}
$typolinkConfiguration = $conf;
- $typolinkConfiguration['parameter'] = $parts[0];
+ $typolinkConfiguration['parameter'] = $scheme . $parts[0];
$textstr .= $this->typoLink($linktxt, $typolinkConfiguration) . $parts[1];
} else {
$textstr .= $scheme . $textpieces[$i];
Updated by Gerrit Code Review over 2 years ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74181
Updated by Gerrit Code Review over 2 years ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74181
Updated by Gerrit Code Review over 2 years ago
Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74181
Updated by Simon Schaufelberger over 2 years ago
- Related to Task #97444: Add unit tests for mailto_makelinks added
Updated by Gerrit Code Review over 2 years ago
Patch set 4 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74181
Updated by Gerrit Code Review over 2 years ago
Patch set 5 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74181
Updated by Gerrit Code Review over 2 years ago
Patch set 1 for branch 11.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74310
Updated by Anonymous over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 0a757e066fe15879a178b4eef64e7100cc7776be.
Updated by Gerrit Code Review over 2 years ago
- Status changed from Resolved to Under Review
Patch set 2 for branch 11.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74310
Updated by Gerrit Code Review over 2 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/+/74311
Updated by Anonymous over 2 years ago
- Status changed from Under Review to Resolved
Applied in changeset 23c553784ecccdfb58e1aca160dc546a2535d0e7.
Updated by Gerrit Code Review over 2 years ago
- Status changed from Resolved to Under Review
Patch set 2 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/+/74311
Updated by Stefan Bürk over 2 years ago
- Related to Bug #97457: Use proper LinkFactory setup for newly added unit tests added
Updated by Riccardo De Contardi over 1 year ago
- Status changed from Under Review to Closed
As far as I can see the patch has been abandoned on version 10 and is already merged on 11 => closing it
If you think that this is the wrong decision, please reopen it or ping me and I'll do it.