You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Carsten Ziegeler (JIRA)" <ji...@apache.org> on 2017/03/24 11:18:41 UTC

[jira] [Resolved] (SLING-6064) Redirect servlet should encode url for redirecting

     [ https://issues.apache.org/jira/browse/SLING-6064?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Carsten Ziegeler resolved SLING-6064.
-------------------------------------
    Resolution: Fixed
      Assignee: Carsten Ziegeler

Fixed in rev 1788408

> Redirect servlet should encode url for redirecting
> --------------------------------------------------
>
>                 Key: SLING-6064
>                 URL: https://issues.apache.org/jira/browse/SLING-6064
>             Project: Sling
>          Issue Type: Bug
>          Components: Servlets
>    Affects Versions: Servlets Get 2.1.18
>            Reporter: Carsten Ziegeler
>            Assignee: Carsten Ziegeler
>             Fix For: Servlets Get 2.1.24
>
>
> The RedirectServlet is directly setting the location header (wondering why sendRedirect is not used instead?) however it is not encoding the URL (calling encodeRedirectURL). Therefore if query parameters are appended these are not encoded. According to the servlet spec, the url should be encoded before being passed to sendRedirect. I would assume the same applies to setting the header as it goes in there unmodified



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)