Uploaded image for project: 'Commons Configuration'
  1. Commons Configuration
  2. CONFIGURATION-221

Allow system properties in config.xml

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.2
    • Fix Version/s: 1.3
    • Component/s: None
    • Labels:
      None
    • Environment:

      All

      Description

      The attached patch allows system properties in the filename attributes of a config.xml file used in the factory.

      The patch is very simple, since commons-digester parses the file and has a built in capability to do the property substitution.

      The dependency of commons-configuration to commons-digester goes up from version 1.5 to 1.6. The most recent version 1.7 of commons-digester is not necessary.

      The patch can easily be ported to 1.3 and trunk.

      Use case: often the config.xml files are contained in a distribution unit like a war file, that should not be manipulated after packaging. The path names in config.xml need some way of making them externally variable so that the packaging unit can be adoped to a special environment without repackaging. By allowing system properties in the file names, this can be done easily.

      The implementation is simple, short and low risk.

      It would be nice, if you considered the patch for inclusion in the next release.

        Attachments

          Activity

            People

            • Assignee:
              oheger Oliver Heger
              Reporter:
              rjung Rainer Jung
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: