Skip to content

Commit

Permalink
Merge pull request #12 from Trihydro/optional-tls-auth/default-to-true
Browse files Browse the repository at this point in the history
Added default values of "true" for TLS & AUTH toggles
  • Loading branch information
dmccoystephenson authored Sep 5, 2024
2 parents e67b850 + 94c8e51 commit e4d26aa
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions services/api/src/contact_support.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,8 @@ def __init__(self):
self.CSM_TARGET_SMTP_SERVER_PORT = int(
os.environ.get("CSM_TARGET_SMTP_SERVER_PORT")
)
self.CSM_TLS_ENABLED = os.environ.get("CSM_TLS_ENABLED")
self.CSM_AUTH_ENABLED = os.environ.get("CSM_AUTH_ENABLED")
self.CSM_TLS_ENABLED = os.environ.get("CSM_TLS_ENABLED", "true")
self.CSM_AUTH_ENABLED = os.environ.get("CSM_AUTH_ENABLED", "true")
self.CSM_EMAIL_APP_USERNAME = os.environ.get("CSM_EMAIL_APP_USERNAME")
self.CSM_EMAIL_APP_PASSWORD = os.environ.get("CSM_EMAIL_APP_PASSWORD")

Expand All @@ -55,7 +55,7 @@ def __init__(self):
if not self.CSM_TARGET_SMTP_SERVER_PORT:
logging.error("CSM_TARGET_SMTP_SERVER_PORT environment variable not set")
abort(500)

if self.CSM_AUTH_ENABLED == "true":
if not self.CSM_EMAIL_APP_USERNAME:
logging.error("CSM_EMAIL_APP_USERNAME environment variable not set")
Expand Down

0 comments on commit e4d26aa

Please sign in to comment.