[ 
https://issues.apache.org/jira/browse/SLING-7548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16618883#comment-16618883
 ] 

Konrad Windszus edited comment on SLING-7548 at 9/18/18 10:32 AM:
------------------------------------------------------------------

The filter doesn't distinguish between absolute paths and absolute URLs 
(including host). The latter could in theory also point to an internal 
resource. Right now the extension is only added if the path part of the URL is 
not empty (see SLING-5632). Therefore in the above example you don't get the 
extension for {{[https://www.adobe.com|https://www.adobe.com/fr/]}} (a URL 
without a part) but for the other one you do. I think this is correct 
behaviour. Why do you expect that only URLs not starting with a host are 
modified through that filter?


was (Author: kwin):
The filter doesn't distinguish between absolute paths and absolute URLs 
(including host). The latter could in theory also point to an internal 
resource. Right now the extension is only added if the path part of the URL is 
not empty. Therefore in the above example you don't get the extension for 
{{[https://www.adobe.com|https://www.adobe.com/fr/]}} (a URL without a part) 
but for the other one you do. I think this is correct behaviour. Why do you 
expect that only URLs not starting with a host are modified through that filter?

> HTL url manipulation for external link without extension
> --------------------------------------------------------
>
>                 Key: SLING-7548
>                 URL: https://issues.apache.org/jira/browse/SLING-7548
>             Project: Sling
>          Issue Type: Bug
>            Reporter: Pony Soeung
>            Priority: Major
>
> For $\{'[https://www.adobe.com|https://www.adobe.com/fr/]' @ extension 
> ='html'}, the url is untouched. OK
> For $\{'[https://www.adobe.com/fr/]' @ extension ='html'}, '.html' is added 
> to the url. KO



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to