You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-dev@portals.apache.org by "Neil Griffin (JIRA)" <ji...@apache.org> on 2019/04/03 22:18:00 UTC

[jira] [Closed] (PLUTO-766) PageAdminPortlet.addPage(ActionRequest) does not check if the portlet context path already contains a leading forward-slash

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

Neil Griffin closed PLUTO-766.
------------------------------
    Resolution: Fixed

Fixed in commit [39f49aadba768bcfe37bd5e92b5886951a32740e|https://github.com/apache/portals-pluto/commit/39f49aadba768bcfe37bd5e92b5886951a32740e]. Many thanks to Hero Wanders for the fix.

> PageAdminPortlet.addPage(ActionRequest) does not check if the portlet context path already contains a leading forward-slash
> ---------------------------------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-766
>                 URL: https://issues.apache.org/jira/browse/PLUTO-766
>             Project: Pluto
>          Issue Type: Bug
>          Components: portlets-admin
>    Affects Versions: 3.0.0, 3.0.1
>            Reporter: Neil Griffin
>            Assignee: Neil Griffin
>            Priority: Major
>             Fix For: 3.0.2
>
>
> The following fragment of code in the {{PageAdminPortlet.addPage(ActionRequest)}} method does not check if the portlet context path already contains a leading forward-slash:
> {code:java|title=PageAdminPortlet.java}
> String contextPath = applicationName;
> if (contextPath.length() > 0)
> {
>     contextPath = "/" + contextPath;
> }
> {code}
> Because of this, the {{contextPath}} might end up with double forward-slashes.



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