Skip to content

Commit 95976f4

Browse files
committed
Don't load main config twice
Before, the main_config (e.g. 'configuration.py' or 'ldap_config.py') were loaded twice. The first load was hard-coded and correct. The second load was during the discovery phase for dynamic configurations. This has now been fixed. During the discovery for dynamic configurations, the main_config file is now excluded. Thanks @tobiasge for discovering this bug.
1 parent 009eb1f commit 95976f4

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

docker/configuration.docker.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,9 @@ def read_configurations(config_module, config_dir, main_config):
4545
if not f.name.endswith(".py"):
4646
continue
4747

48+
if f.name == f"{main_config}.py":
49+
continue
50+
4851
if f.name == f"{config_dir}.py":
4952
continue
5053

0 commit comments

Comments
 (0)