[Bug 62584] New: Removing '/./' in redirected URLs (Wicket-Feature)

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62584

            Bug ID: 62584
           Summary: Removing '/./' in redirected URLs (Wicket-Feature)
           Product: JMeter
           Version: 4.0
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: HTTP
          Assignee: issues@jmeter.apache.org
          Reporter: peter@gruener-it.at
  Target Milestone: ---

In addition to Bug 49083 does Wicket not only produce collapsing URLs like

"app/path/../path2/" 

but also relative paths like

"app/wicket/./login?0"

in redirected URLs.

Current Browsers fix URLs with "/./" by removing "/./" but not JMeter.

In this Bug the problem with relative paths shall be fixed by replacing "/./"
by "/" in the URLs getting by redirection.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62584] Removing '/./' in redirected URLs (Wicket-Feature)

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62584

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62584] Removing '/./' in redirected URLs (Wicket-Feature)

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62584

--- Comment #1 from peter@gruener-it.at ---
Created attachment 36063
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=36063&action=edit
Patch

Patch for this Bug

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62584] Removing '/./' in redirected URLs (Wicket-Feature)

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62584

Felix Schumacher <fe...@internetallee.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #2 from Felix Schumacher <fe...@internetallee.de> ---
If you look at ConversionUtils#removeSlashDotDot, you will see, that it does a
lot more than simply removing '/..' from the complete URL.

Maybe it would be a good idea to factor out the common logic of /.. and /. into
a new method and delegate the old one and the new one (you would have to
create, too) to it.

And by the way. String#replaceAll takes a regex as first command and '.' has a
special meaning in regex, so it is probably a good idea to add test cases for
the change :) It should be easier, if you take the route via ConversionUtils.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62584] Removing '/./' in redirected URLs (Wicket-Feature)

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62584

peter@gruener-it.at changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |peter@gruener-it.at
                 OS|                            |All

-- 
You are receiving this mail because:
You are the assignee for the bug.