diff --git a/server_environment/readme/newsfragments/265.bugfix b/server_environment/readme/newsfragments/265.bugfix new file mode 100644 index 000000000..4e074060d --- /dev/null +++ b/server_environment/readme/newsfragments/265.bugfix @@ -0,0 +1 @@ +remove interpolation in the configparser diff --git a/server_environment/server_env.py b/server_environment/server_env.py index e1194d165..bfc7b7dea 100644 --- a/server_environment/server_env.py +++ b/server_environment/server_env.py @@ -155,7 +155,7 @@ def _load_config_from_env(config_p): def _load_config(): """Load the configuration and return a ConfigParser instance.""" - config_p = configparser.ConfigParser() + config_p = configparser.ConfigParser(interpolation=None) # options are case-sensitive config_p.optionxform = str diff --git a/server_environment/tests/test_server_environment.py b/server_environment/tests/test_server_environment.py index 822d4a7e0..d97076792 100644 --- a/server_environment/tests/test_server_environment.py +++ b/server_environment/tests/test_server_environment.py @@ -72,7 +72,7 @@ def test_value_retrieval(self): with self.set_config_dir("testfiles"): parser = server_env._load_config() val = parser.get("external_service.ftp", "user") - self.assertEqual(val, "testing") + self.assertEqual(val, "testing%") val = parser.get("external_service.ftp", "host") self.assertEqual(val, "sftp.example.com") diff --git a/server_environment/tests/testfiles/testing/base.conf b/server_environment/tests/testfiles/testing/base.conf index 46da78487..eeed2736f 100644 --- a/server_environment/tests/testfiles/testing/base.conf +++ b/server_environment/tests/testfiles/testing/base.conf @@ -2,4 +2,4 @@ odoo_test_option = Set in config file for testing env [external_service.ftp] -user = testing +user = testing%