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

WebXmlUtil.areFilterDispatchersConfiguredProperly method is overly strict

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0, 3.6.4, 3.6.5
    • Component/s: core
    • Labels:
      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.

        Attachments

          Activity

            People

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

              Dates

              Created:
              Updated:
              Resolved:
              Date of First Response: