Bug 58988 - $ escaping for rewrite
Summary: $ escaping for rewrite
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 9
Classification: Unclassified
Component: Catalina (show other bugs)
Version: unspecified
Hardware: PC All
: P2 major (vote)
Target Milestone: -----
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-02-10 14:22 UTC by Remy Maucherat
Modified: 2016-02-11 16:34 UTC (History)
0 users



Attachments
Let backslashes escape characters (4.69 KB, patch)
2016-02-10 14:53 UTC, Felix Schumacher
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Remy Maucherat 2016-02-10 14:22:38 UTC
The following escaping behavior should be implemented:
https://httpd.apache.org/docs/2.2/mod/mod_rewrite.html#quoting
Comment 1 Felix Schumacher 2016-02-10 14:53:29 UTC
Created attachment 33544 [details]
Let backslashes escape characters

This will enable escaping (quoting) by using an backslash.

Apart from this, it will enable escaping the percent sign by using %%. It also fixes a bug, when % was not followed by a digit or a curly brace.

The functionality for %% should probably not be added.

Documentation for the quotation feature is missing, too.
Comment 2 Remy Maucherat 2016-02-10 14:55:56 UTC
Normally % should be escaped with \% according to the documentation, not anything else.
Comment 3 Remy Maucherat 2016-02-11 12:54:26 UTC
This looks fixed by r1729730
Comment 4 Felix Schumacher 2016-02-11 16:34:40 UTC
Fixed in 9.0.0.M4 and 8.0.33.