[MAGNOLIA-1802] include of servlets registered in the servlet filter are ignored Created: 26/Oct/07  Updated: 03/Dec/13  Resolved: 03/Dec/13

Status: Closed
Project: Magnolia
Component/s: core
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Major
Reporter: Philipp Bärfuss Assignee: Unassigned
Resolution: Outdated Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: JPEG File probelm.jpg     PNG File solutions1&2.png    
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   

Since the servlets are loaded in a filter, the include does not work accordingly. But since the filter chain is not executed a second time the included servlet isn't executed.



 Comments   
Comment by Philipp Bracher [ 26/Oct/07 ]

attached a diagram illustrating the probelme

Comment by Philipp Bracher [ 26/Oct/07 ]

solutions
1) filter dispatch include

  • filter chain reexecuted
  • bypass for filter (on include)

2) wrap the request (already done)

  • the returned Requestdispatcher handles servlets registered at the filter
Comment by Philipp Bracher [ 26/Oct/07 ]

Other solutions are:

3) put servlets to web.xml

4) ignore it and find a different solution for the acl dialog

  • other servlets (from end user) are registered in web.xml
Comment by Philipp Bracher [ 26/Oct/07 ]

For the acl dialog I have transformed the page into a control (anyway a bit cleaner)

But the main issue stays but will be solved the day we change the filter behavior MAGNOLIA-1698. On that day we will implement solution 1)

Comment by Philipp Bracher [ 26/Oct/07 ]

I made a more general task out of it and scheduled it for orange

Generated at Mon Feb 12 03:30:28 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.