Geronimo
  1. Geronimo
  2. GERONIMO-1683

Web app context-root should trim whitespace

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.0
    • Fix Version/s: 1.1
    • Component/s: web
    • Security Level: public (Regular issues)
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      If you have an element like this:

      <context-root>
      /something
      </context-root>

      Then the whitespace is included in the context root – it becomes "return space space /something" instead of just "/something". We should trim the whitespace.

      1. 1683-context-root-trim-new.patch
        14 kB
        Erin Mulder
      2. 1683-context-root-trim.patch
        14 kB
        Erin Mulder

        Activity

        Hide
        Aaron Mulder added a comment -

        patch applied, thanks!

        Show
        Aaron Mulder added a comment - patch applied, thanks!
        Hide
        Erin Mulder added a comment -

        Updated patch. (Slightly cleaner, working test case.)

        Show
        Erin Mulder added a comment - Updated patch. (Slightly cleaner, working test case.)
        Hide
        Erin Mulder added a comment -

        The attached patch includes fixes this issue in tomcat-builder and jetty-builder. Includes tests.

        Show
        Erin Mulder added a comment - The attached patch includes fixes this issue in tomcat-builder and jetty-builder. Includes tests.

          People

          • Assignee:
            Aaron Mulder
            Reporter:
            Aaron Mulder
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development