Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-2319

WebXmlUtil.areFilterDispatchersConfiguredProperly method is overly strict

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 4.0, 3.6.4, 3.6.5
    • None
    • core
    • None

      This method is called within the info.magnolia.module.delta.WebXmlConditionsUtil.filterMustBeRegisteredWithCorrectDispatchers method, to test that there are REQUEST and FORWARD dispatchers configured in the web.xml file before starting/installing the core module.

      However, in our case we have added the INCLUDE dispatcher in order to enable inclusion of magnolia content into other web apps. Since this test only passes if there are only these two dispatchers configured (with optional "ERROR"), it fails even if these 2 dispatchers are present along with the INCLUDE. This prevents the core module from installing on bootstrapped startup.

      I think it should only test the mandatory presence of REQUEST and FORWARD, and not validate additional dispatchers.

        Acceptance criteria

              gjoseph Magnolia International
              dallasvaughan Dallas Vaughan
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD