Bug 999961

Summary: When adding rewrite configuration for virtual server no reload is required, but the setting is not active till reload is actually performed
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Radim Hatlapatka <rhatlapa>
Component: WebAssignee: Rémy Maucherat <rmaucher>
Status: CLOSED CURRENTRELEASE QA Contact: Michael Cada <mcada>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.1.1CC: brian.stansberry, pslavice
Target Milestone: DR4   
Target Release: EAP 6.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-08-19 12:38:42 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Radim Hatlapatka 2013-08-22 13:05:38 UTC
Description of problem:
When setting rewrite for virtual server no reload is required, but when accessing an URL which should be rewritten, the rewrite is not effective until reload or restart of the server is performed

e.g.
[standalone@localhost:9999 /] /subsystem=web/virtual-server=default-host/rewrite=test:add(flags=NC, pattern=(.*)/toberewritten/(.*), substitution=$1/rewritten/$2)
{"outcome" => "success"}


Steps to Reproduce:
1. add rewrite setting (e.g. via CLI /subsystem=web/virtual-server=default-host/rewrite=test2:add(flags=F, pattern=(.*), substitution=-)
 substitution=$1/rewritten/$2)
2. see that operation doesn't require reload
3. go to http://localhost:8080/ - the page is accessible (due rewrite setting it shouldn't be)
4. reload server (via CLI /:reload)
5. go to http://localhost:8080/ - the access is forbidden => rewrite rule finally applied


Actual results:
Operation for adding rewrite configuration on virtual host doesn't prompt that reload is required even though it is

Expected results:
Either apply the setting without reload requirement, or prompt that it is actually required

Comment 2 Martin Velas 2014-08-05 12:20:11 UTC
Issue is still valid for the EAP 6.3.0.ER10 bits.

Comment 3 Rémy Maucherat 2014-09-11 09:56:18 UTC
https://github.com/jbossas/jboss-eap/pull/1668

Comment 4 Radim Hatlapatka 2014-10-15 14:17:26 UTC
This is already merged and in the product.

Verified in EAP 6.4.0.DR4