[MAGNOLIA-2085] When registering a bypass filter of class URIStartsWithVoter causes the server to die Created: 12/Mar/08 Updated: 23/Jan/13 Resolved: 12/Mar/08 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | 3.5.4 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Critical |
| Reporter: | Ryan Gardner | Assignee: | Philipp Bärfuss |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
| Date of First Response: |
| Description |
|
All I did was follow the instructions on the wiki exactly as they are written... After I created the nodeData for the class of the bypass filter - and before I could put any more information in it - the server died with this message: http://wiki.magnolia.info/display/WIKI/HowTo+Setup+Custom+404+Handler Mar 11, 2008 5:59:42 PM org.apache.catalina.core.StandardWrapperValve invoke I couldn't do anything to get it back at that point, and had to restore the app from a backup. It was as if the filter was being immediately put into effect, before it was even configured fully. Even if it were configured completely wrong, there is really no excuse for the server to completely die because of a bad user input. |
| Comments |
| Comment by Jan Haderka [ 12/Mar/08 ] |
|
As with all other configuration changes in magnolia they are applied immediately. Since empty pattern is valid pattern for the voter it is considered configured even before you enter desired pattern. It would be nice to have temp space for configuration and activate all changes later, but not possible with current UI. |