ConfigReader regex doubt

[lukaszsus]

Hello,

Reading the code of magda, I have noticed a line that probably doesn't work as expected to work. The line is magda/config_reader.py:71:

declared_variables = list(set(re.findall(r'\${(\w+)}', config_str)))

I am quite sure that this regex also finds parameters that are commented (with #) and I think it is undesired behavior.