Tijmen commented on SLING-7493:

In addition to the '.aspx' extension that gets rewritten to an '.html' 
extension, the current implementation also add the '.html' extension to 
external links if those links contain a forward slash:

 * [https://www.twitter.com|https://www.google.com/] is unmodified, as expected.
 * [https://www.twitter.com/TheASF] gets an '.html' extension, this is often 
unwanted behavior; [https://www.twitter.com/TheASF.html].
 * [https://www.twitter.com/TheASF.aspx] loses its existing extension (it is 
replaced with '.html'), which is often unwanted behavior; 

> HTL must leave external urls untouched
> --------------------------------------
>                 Key: SLING-7493
>                 URL: https://issues.apache.org/jira/browse/SLING-7493
>             Project: Sling
>          Issue Type: Improvement
>          Components: Scripting
>    Affects Versions: Scripting HTL Engine 1.0.48-1.3.1
>            Reporter: Feike Visser
>            Priority: Major
> In the current implementation of @ extension option of HTL, also external 
> urls are changed.
> It is very common to have a list with both internal and external urls.
> <a href="${ item.link @ extension='html'}>...</a>
> Now when you have an external url with .aspx, it also gets replaced with 
> .html.
> I know this is in the spec, but I can't think of usecases where you want to 
> have this.

This message was sent by Atlassian JIRA

Reply via email to