DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-11-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 --- Comment #20 from Mark Thomas 2010-11-21 15:50:54 EST --- (In reply to comment #19) > Why not porting this feature to tc7 ? Servlet 3.0 already supports it. > even if servlet 3 apps can do it through web.xml, we should think of servlet

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-11-20 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 --- Comment #19 from sylvain.laur...@gmail.com 2010-11-20 14:08:30 EST --- Why not porting this feature to tc7 ? even if servlet 3 apps can do it through web.xml, we should think of servlet 2 applications that are deployed in tc7... -- Con

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-11-20 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 sylvain.laur...@gmail.com changed: What|Removed |Added CC||sylvain.laur...@gmail.co

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-10-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-10-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 --- Comment #16 from Wesley 2010-10-10 19:28:52 EDT --- (In reply to comment #15) > Created an attachment (id=26135) --> (https://issues.apache.org/bugzilla/attachment.cgi?id=26135) [details] > Updated patch > > I have reviewed the patch

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-10-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 Mark Thomas changed: What|Removed |Added Attachment #26018|0 |1 is obsolete|

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-09-27 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 --- Comment #14 from Wesley 2010-09-27 13:57:52 EDT --- (In reply to comment #13) > Created an attachment (id=26018) --> (https://issues.apache.org/bugzilla/attachment.cgi?id=26018) [details] > V3 with tabs fixed and parameter name on Con

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-09-11 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 Wesley changed: What|Removed |Added Attachment #25961|0 |1 is obsolete|

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-09-11 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 --- Comment #12 from Mark Thomas 2010-09-11 13:39:50 EDT --- (In reply to comment #11) > Em do you have a style guide? If theres any code formatters I can use for > eclipse that would be great. Then I can just use the format shortcut. 'fra

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-09-11 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 --- Comment #11 from Wesley 2010-09-11 13:27:48 EDT --- (In reply to comment #10) > - use 4 spaces rather than tabs Em do you have a style guide? If theres any code formatters I can use for eclipse that would be great. Then I can just us

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-09-11 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 --- Comment #10 from Mark Thomas 2010-09-11 13:13:15 EDT --- Functionally, the patch looks good to me and I'd propose it for Tomcat if it wasn't for some minor cosmetic issues: - use 4 spaces rather than tabs - setDisableURLRewriting(boolea

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-09-09 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 --- Comment #9 from Wesley 2010-09-09 13:57:54 EDT --- >It was a good initiative and I'm sure we will have a look at the patch. Please >be patient though. If you don't see any progress (comments in the Bugzilla >issue), then it is fine t

DO NOT REPLY [Bug 49811] [PATCH] Disable UrlRewriting

2010-09-09 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49811 Wesley changed: What|Removed |Added Summary|PATCH Custom attribute to |[PATCH] Disable |not en