URL Rewrites fire incorrect on URL containing a space

Description

Start a server with rewrites and place a static file like an image in a sub folder that has a space in it and hit that image from the URL. The rewrite rule will kick in and try to add /index.cfm because the image path won't be found on disk. Enabling the URL rewrite log and starting the server with --trace will reveal that the file path in use contains the %20 instead of a space. Tuckey seems to be mishandling the space. Possibly not casting a URL to a URI Path.

Activity

Show:
Brad Wood
November 1, 2019, 8:19 PM

Did you make changes to the rewrite library to support this, or was this a fix Denny already had in place that you simply tested?

Miguel Mathus
November 4, 2019, 3:29 PM

This was a change made by Denny on MappedResourceManager class on Method getResource(path), I do a test creating a new site with a sub folder and an image, and try to reach it

Brad Wood
November 4, 2019, 3:43 PM

Thx, so to review, you are in the process of testing this, or you have already tested it and found it to pass? If this ticket is not yet tested, don’t mark it resolved! cc/

Miguel Mathus
November 15, 2019, 2:59 PM

Hi , I have already tested this trying to reach an static image with a space in the name, and it’s working properly, cc

Checked 15/11/19

Brad Wood
November 16, 2019, 12:00 AM

Thanks!

Assignee

Miguel Mathus

Reporter

Brad Wood

Labels

None

Affects versions

Fix versions

Priority

Major

Components

Configure