Bugfix for multiple values of a key in a Config file - apache#9102, apache#12689

[deepthi912]

Fix #9102 #12689

Overriding the existing values with the last value in the list instead of adding them as a list of values in the Config File.

After Fix: controller.data.dir is taken as the last value.

[Jackie-Jiang]

This will cause problem for real array values. I think the core problem is that there is no way to differentiate key with single value vs array

[xiangfu0]

This will cause problem for real array values. I think the core problem is that there is no way to differentiate key with single value vs array

I didn't recall that there is an explicitly method to merge duplicate keys into one comma separated array.

Is this really used somewhere or possibly we can error it out? Feeling it's causing more confusion than the convenient.

[deepthi912]

@Jackie-Jiang Yes, this fix was made assuming there wouldn't be an array of values for a key in a config file. However since there can be such scenarios, it is difficult to differentiate between real array values and multiple keys in a config when set through different sources. Only if Apache Configuration makes some upgrade removing the existing behaviour, this could be fixed