Clean up error output when validating settings values (#265)

docs/source/overview/installation.rst

@@ -44,7 +44,7 @@ The :doc:`file_systems` page provides an overview of supported file system types
        ],
        "email_from": "[email protected]",
        "email_domain": "@domain.com",
-       "email_admins": ["[email protected]"]
+       "admin_emails": ["[email protected]"]
    }
 
 Once the application has been configured, you can check the configuration file is valid by running:

quota_notifier/cli.py

@@ -11,6 +11,7 @@
 
 import logging
 import logging.config
+import sys
 from argparse import ArgumentParser
 from pathlib import Path
 from smtplib import SMTP
@@ -116,7 +117,7 @@ def _configure_logging(cls, console_log_level: int) -> None:
                     'level': 'CRITICAL',
                     'mailhost': ApplicationSettings.get('smtp_host'),
                     'fromaddr': ApplicationSettings.get('email_from'),
-                    'toaddrs': ApplicationSettings.get('email_admins'),
+                    'toaddrs': ApplicationSettings.get('admin_emails'),
                     'subject': 'Quota Notifier - Admin Notification'
                 }
             },
@@ -167,7 +168,13 @@ def run(cls, validate: bool = False, verbosity: int = 0, debug: bool = False) ->
         """
 
         # Configure application settings
-        cls._load_settings(force_debug=debug)
+        try:
+            cls._load_settings(force_debug=debug)
+
+        except Exception as e:
+            print(e)
+            sys.exit(0)
+
         if validate:
             return
 

quota_notifier/settings.py

@@ -7,7 +7,6 @@
 Module Contents
 ---------------
 """
-
 import logging
 from pathlib import Path
 from tempfile import NamedTemporaryFile
@@ -180,7 +179,7 @@ class SettingsSchema(BaseSettings):
         description=('String to append to usernames when generating user email addresses. '
                      'The leading `@` is optional.'))
 
-    email_admins: List[str] = Field(
+    admin_emails: List[str] = Field(
         title='Administrator Emails',
         default=[],
         description='Admin users to contact when the application encounters a critical issue.'
@@ -237,16 +236,7 @@ def set_from_file(cls, path: Path) -> None:
             path: Path to load settings from
         """
 
-        logging.debug(f'Looking for settings file: {path.resolve()}')
-
-        try:
-            cls._parsed_settings = SettingsSchema.parse_file(path)
-
-        except Exception:
-            logging.error('settings file is invalid')
-            raise
-
-        logging.info(f'Loaded settings from file: {path.resolve()}')
+        cls._parsed_settings = SettingsSchema.model_validate_json(path.read_text())
 
     @classmethod
     def set(cls, **kwargs) -> None: