PR#866: consolidate ConfigParser usage -

koji

#866 consolidate ConfigParser usage

Closed Jan 2, 2020 by tkopecek. Opened Mar 29, 2018 by tkopecek.

tkopecek/koji issue865 into master

tkopecek commented Mar 29, 2018

Fixes: https://pagure.io/koji/issue/865

rebased onto 6c953a414f2b8ddb0ba489b6b43b4fea73e9c529

Mar 29, 2018

tkopecek commented Mar 29, 2018

Is there a specific reason, why RawConfigParser was used in kojixmlrpc.py? Other places seems to be ok with SafeConfigParser, but not sure if there was reason here to be more strict.

mikem commented Apr 3, 2018

The other config parsers support a substitution syntax that we do not use, and that can break certain values in the config (notably LogFormat settings)

tkopecek commented Apr 11, 2018

So, do you think it is ok to leave RawconfigParser in kojixmlrpc and all other change to SafeConfigParser?

tkopecek commented Jan 2, 2020

Dropping as obsoleted by #1296

Pull-Request has been closed by tkopecek

Jan 2, 2020