Skip to content

Commit 78286a9

Browse files
committed
Modify upon review
Signed-off-by: Alina Buzachis <[email protected]>
1 parent e7986f8 commit 78286a9

File tree

2 files changed

+17
-22
lines changed

2 files changed

+17
-22
lines changed

core/apps.py

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,23 @@ class CoreConfig(AppConfig):
1414
name = "core"
1515

1616
def ready(self) -> None:
17+
# List of required AAP variables
18+
required_vars = [
19+
"AAP_URL",
20+
"AAP_USERNAME",
21+
"AAP_PASSWORD",
22+
]
23+
24+
# Check that each required setting is defined
25+
missing = [var for var in required_vars if not getattr(settings, var, None)]
26+
if missing:
27+
logger.error(
28+
f"Missing required configuration variables: {', '.join(missing)}"
29+
)
30+
raise RuntimeError(
31+
f"Required AAP variable not defined: {', '.join(missing)}. "
32+
)
33+
1734
# Validate AAP_URL
1835
try:
1936
settings.AAP_URL = validate_url(settings.AAP_URL)

pattern_service/settings/aap.py

Lines changed: 0 additions & 22 deletions
This file was deleted.

0 commit comments

Comments
 (0)