Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.1
    • Fix Version/s: 5.2-milestone-1
    • Component/s: Velocity
    • Labels:
      None
    • Tests:
      Unit
    • Similar issues:

      Description

      Right now, we don't have any tool to sanitize some query string that would be put inserted into an url. As this is needed in many places, we could add the following method to escapetool:

      String url(Map<String, ?>)

      that would create a safe and correctly escaped query String from a map of parameters.
      See the discussion on https://github.com/xwiki/xwiki-platform/pull/134 for more details.

        Activity

        Hide
        Marius Dumitru Florea added a comment -

        Pull request applied. Thanks! Can you please add some words in the release notes? Thanks again

        Show
        Marius Dumitru Florea added a comment - Pull request applied. Thanks! Can you please add some words in the release notes? Thanks again
        Hide
        Thomas Delafosse added a comment -

        I added a short description in the 5.2M1 release notes. Thanks !

        Show
        Thomas Delafosse added a comment - I added a short description in the 5.2M1 release notes. Thanks !

          People

          • Assignee:
            Thomas Delafosse
            Reporter:
            Thomas Delafosse
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Date of First Response: