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

OverrideCombiner does not work properly when key contains list

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Format
    • Labels:
      None

      Description

      It seems that the OverrideCombiner does not work properly in some cases when key contains a list. The problem is best illustrated with an example:

       

      In your Base Config File you following key values:

      idp.auth.methods = NO-BANKID, NO-BANKID-MOBILE

      And in our Specialty configuration, the one that overrides duplicates in the base config file contains following key:

      idp.auth.methods.1443 = SE-BANKID, SE-BANKID-MOBILE

       

      The expected result after combining these two configuration files would be:

       

      idp.auth.methods = NO-BANKID, NO-BANKID-MOBILE

      idp.auth.methods.1443 = SE-BANKID, SE-BANKID-MOBILE

       

      However the actual result of current implementation is as follows:

      idp.auth.methods.1443 = SE-BANKID, SE-BANKID-MOBILE

       

      So even though that the config file that overrides the base configuration file does not have any overriding configuration for idp.auth.methods. The config in the base config connected to key idp.autyh.methods are not added to the combined configuration.

       

      If this is not a bug but a feature, then i would like to know which combiner i should use as it is not obvious from the documentation. The objective is to be able to have one file that contains extra configuration which also can include configuration keys that should override configuration keys from the base configuration file.

       

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              gisle.grimen@evry.com Gisle Grimen
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: