Pluto
  1. Pluto
  2. PLUTO-417

PortletResponseImpl doesn't allow wsrp rewrite urls

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1.4, 1.2.0
    • Fix Version/s: 1.1.5, 2.0.0
    • Component/s: portlet container
    • Labels:
      None

      Description

      WSRP producers may create urls in a specific format, designed to be re-written by the consumer.

      See 10.2.1 of the WSRP spec. These urls are delimited by the start token 'wsrp_rewrite?' and the end token '/wsrp_rewrite'. The current implementation of the PortletResponseImpl doesn't allow these WSRP rewrite urls to be encoded :

      public String encodeURL(String path) {
      if (path.indexOf("://") == -1 && !path.startsWith("/"))

      { throw new IllegalArgumentException( "Only absolute URLs or full path URIs are allowed. Offending url: [" + path + "]" ); }

        Activity

        Hide
        Elliot Metsger added a comment -

        Updated fix versions.

        Applied to 1.1-286-trunk-merge branch r609273. Had to update the unit test to accommodate 286 cache control and new class hierarchy for PortletResponseImpl.

        Show
        Elliot Metsger added a comment - Updated fix versions. Applied to 1.1-286-trunk-merge branch r609273. Had to update the unit test to accommodate 286 cache control and new class hierarchy for PortletResponseImpl.
        Hide
        Elliot Metsger added a comment -

        Applied to the 1.1.x branch. Still needs to be applied to 1.2.0. I'll wait for the 286 merge work to complete before applying to trunk.

        Show
        Elliot Metsger added a comment - Applied to the 1.1.x branch. Still needs to be applied to 1.2.0. I'll wait for the 286 merge work to complete before applying to trunk.

          People

          • Assignee:
            Elliot Metsger
            Reporter:
            Elliot Metsger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development