Bug 1318006 - [GSS](6.4.z) Rewrite file ResourceConditions always return true
Summary: [GSS](6.4.z) Rewrite file ResourceConditions always return true
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: Web
Version: 6.4.6
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: CR1
: EAP 6.4.8
Assignee: Chao Wang
QA Contact: Radim Hatlapatka
URL:
Whiteboard:
Depends On:
Blocks: eap648-payload 1313271
TreeView+ depends on / blocked
 
Reported: 2016-03-15 18:14 UTC by Aaron Ogburn
Modified: 2019-11-14 07:36 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-01-17 12:36:05 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Knowledge Base (Solution) 2192211 0 None None None 2016-03-15 18:30:00 UTC

Description Aaron Ogburn 2016-03-15 18:14:03 UTC
Description of problem:

The -d, -f, -s rewrite valve file tests do not work.  The ResourceCondition is simply coded to always return true.

Version-Release number of selected component (if applicable):

7.5.12


How reproducible:

Always


Steps to Reproduce:
1.Configure a rewrite valve that's conditioned with -s:

<rewrite name="rule-1" pattern="^/app/(.*)" substitution="/app/redirect.jsp" flags="RL">
    <condition name="condition-1" test="%{REQUEST_PATH}" pattern="!-s"/>
</rewrite>

2. Note that this redirects all requests, but it should only redirect requests to non-existent files.


Actual results:

ResourceCondition always returns true.


Expected results:

ResourceCondition only returns true as expected if the file exists.


Additional info:

Comment 3 Mike McCune 2016-03-28 22:24:40 UTC
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

Comment 4 Rémy Maucherat 2016-03-29 13:45:06 UTC
r2649, for some reason it wasn't ported earlier from Tomcat.

Comment 7 Michael Cada 2016-05-11 13:38:21 UTC
Verified with EAP 6.4.8.CP.CR2

Comment 8 Petr Penicka 2017-01-17 12:36:05 UTC
Retroactively bulk-closing issues from released EAP 6.4 cumulative patches.


Note You need to log in before you can comment on or make changes to this bug.