When UrlUtility::getFrontendUrlWithSite() receives a relative URL (e.g. /powermail/optin/{hash}/{mail}), it returns it unchanged instead of prepending frontendBase. This breaks any code generating URLs in contexts where only relative paths are available (CLI commands, email rendering via StandaloneView), such as powermail double-opt-in confirmation links.
Root cause: PR #890 attempted to fix external URLs being incorrectly rewritten by adding $targetUri->getHost() === '' to the early-return guard. However, this was too broad — relative URLs have no host, so the condition short-circuits on all relative URLs before any rewriting can occur. PR #891 then refined external URL detection with isExternalUrl() and $backendDomains, but kept the faulty empty-host check intact.
Fix: Remove the $targetUri->getHost() === '' condition from the early return and add an empty-host guard to isExternalUrl() so relative URLs are recognized as non-external and proceed to rewriting.
When
UrlUtility::getFrontendUrlWithSite()receives a relative URL (e.g./powermail/optin/{hash}/{mail}), it returns it unchanged instead of prependingfrontendBase. This breaks any code generating URLs in contexts where only relative paths are available (CLI commands, email rendering viaStandaloneView), such as powermail double-opt-in confirmation links.Root cause: PR #890 attempted to fix external URLs being incorrectly rewritten by adding
$targetUri->getHost() === ''to the early-return guard. However, this was too broad — relative URLs have no host, so the condition short-circuits on all relative URLs before any rewriting can occur. PR #891 then refined external URL detection withisExternalUrl()and$backendDomains, but kept the faulty empty-host check intact.Fix: Remove the
$targetUri->getHost() === ''condition from the early return and add an empty-host guard toisExternalUrl()so relative URLs are recognized as non-external and proceed to rewriting.