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

WebXmlUtil.areFilterDispatchersConfiguredProperly method is overly strict

    XMLWordPrintable

Details

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

    Description

      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.

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD