repos/roboluke
3
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Releases 50 Activity

fix(config): always return a list, even if empty, as required by webex_bot

Browse Source
This commit is contained in:
Luke Tainton 2024-11-24 10:41:08 +00:00
parent 6cac9dc9c2
commit b11cc26daa
No known key found for this signature in database
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
app/utils/config.py
View File

@ -47,31 +47,27 @@ class Config:
return os.environ["N8N_WEBHOOK_URL"] return os.environ["N8N_WEBHOOK_URL"]
@property @property
def approved_users(self) -> list | None: def approved_users(self) -> list:
"""Returns a list of approved users.""" """Returns a list of approved users."""
_emails: list[str] = os.environ.get("APPROVED_USERS", "").split(",") _emails: list[str] = os.environ.get("APPROVED_USERS", "").split(",")
_emails: list[str] = [i.strip() for i in _emails if i] _emails: list[str] = [i.strip() for i in _emails if i]
if not _emails: if not _emails:
return None return []
emails = [i for i in _emails if validate_email_syntax(i)] emails = [i for i in _emails if validate_email_syntax(i)]
return emails return emails
@property @property
def approved_rooms(self) -> list | None: def approved_rooms(self) -> list:
"""Returns a list of approved rooms.""" """Returns a list of approved rooms."""
_rooms: list[str] = os.environ.get("APPROVED_ROOMS", "").split(",") _rooms: list[str] = os.environ.get("APPROVED_ROOMS", "").split(",")
rooms: list[str] = [i.strip() for i in _rooms if i] rooms: list[str] = [i.strip() for i in _rooms if i]
if not rooms:
return None
return rooms return rooms
@property @property
def approved_domains(self) -> list | None: def approved_domains(self) -> list:
"""Returns a list of approved domains.""" """Returns a list of approved domains."""
_domains: list[str] = os.environ.get("APPROVED_DOMAINS", "").split(",") _domains: list[str] = os.environ.get("APPROVED_DOMAINS", "").split(",")
domains: list[str] = [i.strip() for i in _domains if i] domains: list[str] = [i.strip() for i in _domains if i]
if not domains:
return None
return domains return domains