Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-4727

Struts 2.5.8 no longer supports the <include> directive in the struts.xml

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Cannot Reproduce
    • Affects Version/s: 2.5.8
    • Fix Version/s: 2.5.10
    • Component/s: Core
    • Labels:
      None

      Description

      in 2.5.5 and most previous versions of Struts 2 you could use the <include file="struts-local-settings.xml" /> directive to separate out constants and other parameters into changsets that could be managed. This appears to have stopped working based on the fact that we're now getting NumberFormatExceptions when trying to parse struts.multipart.maxSize when it's not defined in the struts.xml but instead in the included file. Setting struts.multipart.maxSize directly corrects this.

        Activity

        Hide
        lukaszlenart Lukasz Lenart added a comment -

        Asked on a user group and got confirmation that everything is ok. Feel free to re-open and post more details.

        Show
        lukaszlenart Lukasz Lenart added a comment - Asked on a user group and got confirmation that everything is ok. Feel free to re-open and post more details.
        Hide
        lukaszlenart Lukasz Lenart added a comment -

        adam brin can you post more details? I would like to start preparing a new release

        Show
        lukaszlenart Lukasz Lenart added a comment - adam brin can you post more details? I would like to start preparing a new release
        Hide
        lukaszlenart Lukasz Lenart added a comment -

        Cannot confirm that, the only possible option that could break that was adding support for constant substitution - see WW-4698

        Show
        lukaszlenart Lukasz Lenart added a comment - Cannot confirm that, the only possible option that could break that was adding support for constant substitution - see WW-4698

          People

          • Assignee:
            lukaszlenart Lukasz Lenart
            Reporter:
            abrin adam brin
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development