[MGNLRES-144] Implement new origin-based ResourcesServlet Created: 07/May/15 Updated: 29/Mar/22 Resolved: 28/May/15 |
|
| Status: | Closed |
| Project: | Magnolia Resources Module |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 2.4 |
| Type: | New Feature | Priority: | Major |
| Reporter: | Mikaël Geljić | Assignee: | Magnolia International |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | platform-cell, servlet | ||
| Σ Remaining Estimate: | Not Specified | Remaining Estimate: | Not Specified |
| Σ Time Spent: | Not Specified | Time Spent: | Not Specified |
| Σ Original Estimate: | Not Specified | Original Estimate: | Not Specified |
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||
| Sub-Tasks: |
|
||||||||||||||||||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||||||||||||||||||
| Epic Link: | Streamline working with resources | ||||||||||||||||||||||||||||||||||||||||||||
| Team: | |||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
Should use the new magnolia-resource-loader module from main, and in particular use the LayeredOrigin which implements the unified resource-loading cascade. Is expected to replace ClasspathSpool servlet and be similarly mapped to /.resources. Random thoughts/things to clarify:
Security: What should the servlet be allowed to serve?
Caching
Compatibility options
|
| Comments |
| Comment by Michael Mühlebach [ 18/May/15 ] |
|
Because it would introduce a dependency cycle at the moment, the resources servlet can not replace the class path spool in the install/setup phase (FilterManagerImpl). |
| Comment by Michael Mühlebach [ 18/May/15 ] |
|
The processed resources are out of scope for the moment. They can still be used but over the old NonInstalledResourcesFilter (aka /resources/ path). |
| Comment by Magnolia International [ 23/May/15 ] |
|
Did a bunch of cleanups with Michael. Still reopened for tests, integration tests, and missing update tasks. |
| Comment by Michael Mühlebach [ 27/May/15 ] |
|
Finished the clean up and adapted all tests accordingly. |
| Comment by Magnolia International [ 27/May/15 ] |
|
| Comment by Magnolia International [ 27/May/15 ] |
|
Additionally, I don't think we should release this without any sort of validation (at the very least by content-type, extension, and/or pattern) |
| Comment by Magnolia International [ 27/May/15 ] |
|
Cleaned up, rebased and squashed on feature/
|
| Comment by Michael Mühlebach [ 28/May/15 ] |
|
Moved update task for class path spool servlet to the resources module. |
| Comment by Magnolia International [ 28/May/15 ] |
|
Looks good, cheers.
|
| Comment by Magnolia International [ 29/May/15 ] |
|
Integrated on master, but created |