 | Draft for 3.7, 3.8 or 4.x ?
... |
Rationale
- URI2RepositoryMapping and VirtualURIMapping have very similar goal and functionality (i.e determine which repository node to use based on a URI)
- URI2RepositoryMapping and VirtualURIMapping have very similar interfaces, yet very different implementations and are not working "together"
- VirtualURIMapping need overhauling to fulfill MAGNOLIA-2343@jira
- See Proposal - overhaul and refactoring of AggregationState
- Could provide a way to generate links too (URI2RepositoryMapping already does, but not VirtualURIMapping)
Implementation ideas
- Merging the 2 functionality under one single and clean interface
- Using mechanisms similar to Apache's mod_rewrite: rules are executed one after the other, and each has the possibility to interrupt the chain
Possible pitfalls
- Order of rules if configured under /modules/xyz/urimappings
- RepositoryMappingFilter and AggregatorFilter would be merged - is there any reason why ContentSecurityFilter sits in between these 2 ?
|
|
ContentSecurityFilter checks content security (not only the url) so it is necessarily to resolve workspace and path first (RepositoryMappingFilter)