You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by "Aaron Mulder (JIRA)" <de...@geronimo.apache.org> on 2006/03/03 13:09:52 UTC

[jira] Created: (GERONIMO-1683) Web app context-root should trim whitespace

Web app context-root should trim whitespace
-------------------------------------------

         Key: GERONIMO-1683
         URL: http://issues.apache.org/jira/browse/GERONIMO-1683
     Project: Geronimo
        Type: Bug
  Components: web  
    Versions: 1.0    
    Reporter: Aaron Mulder
     Fix For: 1.1, 1.0.1


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.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


[jira] Updated: (GERONIMO-1683) Web app context-root should trim whitespace

Posted by "Erin Mulder (JIRA)" <de...@geronimo.apache.org>.
     [ http://issues.apache.org/jira/browse/GERONIMO-1683?page=all ]

Erin Mulder updated GERONIMO-1683:
----------------------------------

    Attachment: 1683-context-root-trim.patch

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

> Web app context-root should trim whitespace
> -------------------------------------------
>
>          Key: GERONIMO-1683
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1683
>      Project: Geronimo
>         Type: Bug
>     Security: public(Regular issues) 
>   Components: web
>     Versions: 1.0
>     Reporter: Aaron Mulder
>     Assignee: Aaron Mulder
>     Priority: Critical
>      Fix For: 1.1
>  Attachments: 1683-context-root-trim.patch
>
> 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.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


[jira] Resolved: (GERONIMO-1683) Web app context-root should trim whitespace

Posted by "Aaron Mulder (JIRA)" <de...@geronimo.apache.org>.
     [ http://issues.apache.org/jira/browse/GERONIMO-1683?page=all ]
     
Aaron Mulder resolved GERONIMO-1683:
------------------------------------

    Resolution: Fixed

patch applied, thanks!

> Web app context-root should trim whitespace
> -------------------------------------------
>
>          Key: GERONIMO-1683
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1683
>      Project: Geronimo
>         Type: Bug
>     Security: public(Regular issues) 
>   Components: web
>     Versions: 1.0
>     Reporter: Aaron Mulder
>     Assignee: Aaron Mulder
>     Priority: Critical
>      Fix For: 1.1
>  Attachments: 1683-context-root-trim-new.patch, 1683-context-root-trim.patch
>
> 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.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


[jira] Updated: (GERONIMO-1683) Web app context-root should trim whitespace

Posted by "Aaron Mulder (JIRA)" <de...@geronimo.apache.org>.
     [ http://issues.apache.org/jira/browse/GERONIMO-1683?page=all ]

Aaron Mulder updated GERONIMO-1683:
-----------------------------------

    Fix Version:     (was: 1.2)
      Assign To: Aaron Mulder
       Priority: Critical  (was: Major)

> Web app context-root should trim whitespace
> -------------------------------------------
>
>          Key: GERONIMO-1683
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1683
>      Project: Geronimo
>         Type: Bug
>     Security: public(Regular issues) 
>   Components: web
>     Versions: 1.0
>     Reporter: Aaron Mulder
>     Assignee: Aaron Mulder
>     Priority: Critical
>      Fix For: 1.1

>
> 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.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


[jira] Updated: (GERONIMO-1683) Web app context-root should trim whitespace

Posted by "Erin Mulder (JIRA)" <de...@geronimo.apache.org>.
     [ http://issues.apache.org/jira/browse/GERONIMO-1683?page=all ]

Erin Mulder updated GERONIMO-1683:
----------------------------------

    Attachment: 1683-context-root-trim-new.patch

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

> Web app context-root should trim whitespace
> -------------------------------------------
>
>          Key: GERONIMO-1683
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1683
>      Project: Geronimo
>         Type: Bug
>     Security: public(Regular issues) 
>   Components: web
>     Versions: 1.0
>     Reporter: Aaron Mulder
>     Assignee: Aaron Mulder
>     Priority: Critical
>      Fix For: 1.1
>  Attachments: 1683-context-root-trim-new.patch, 1683-context-root-trim.patch
>
> 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.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


[jira] Updated: (GERONIMO-1683) Web app context-root should trim whitespace

Posted by "Erin Mulder (JIRA)" <de...@geronimo.apache.org>.
     [ http://issues.apache.org/jira/browse/GERONIMO-1683?page=all ]

Erin Mulder updated GERONIMO-1683:
----------------------------------

    Patch Info: [Patch Available]

> Web app context-root should trim whitespace
> -------------------------------------------
>
>          Key: GERONIMO-1683
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1683
>      Project: Geronimo
>         Type: Bug
>     Security: public(Regular issues) 
>   Components: web
>     Versions: 1.0
>     Reporter: Aaron Mulder
>     Assignee: Aaron Mulder
>     Priority: Critical
>      Fix For: 1.1
>  Attachments: 1683-context-root-trim-new.patch, 1683-context-root-trim.patch
>
> 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.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira