Uploaded image for project: 'Wink'
  1. Wink
  2. WINK-216

tolerate empty path segments to support empty PathParams

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.1
    • Common
    • None
    • Patch Available

    Description

      This is a followup to WINK-214.

      Not sure how exactly we want to approach this. There is a message on the user list requesting support for:

      @Path("/testRuns/

      {testRunsParams}

      /testSuites/

      {testSuitesParams}

      ")
      http://localhost:8080/testqueryapi/rest/testRuns/;name=jaro/testSuites/;eyes=blue

      The problem is that no PathParam was specified in the designated locations; we only have matrix params, so these segments are empty when the URI is normalized in UriTemplateProcessor.normalizeUri.

      input --> /testRuns//testSuites/
      output --> /testRuns/testSuites/

      The double slash is correctly converted to single slash. To support this request, we would have to do something special to allow the empty path segment to not be normalized. See attached test patch (which goes on top of WINK-214 patch).

      Attachments

        1. WINK_216.patch
          5 kB
          Michael Rheinheimer

        Activity

          People

            bluk Bryant Luk
            rott Michael Rheinheimer
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment