Merge branch '140-missing-env-error-handling' into 'develop'

Resolve "django.core.exceptions.AppRegistryNotReady: Apps aren't loaded yet. when missing env values"

Closes #140

See merge request funkwhale/funkwhale!112
This commit is contained in:
Eliot Berriot 2018-03-25 21:07:47 +00:00
commit 5efbb72e01
2 changed files with 8 additions and 0 deletions

View File

@ -1,4 +1,5 @@
#!/usr/bin/env python
import django
import os
import sys
@ -7,6 +8,12 @@ sys.path.append(os.path.dirname(os.path.abspath(__file__)))
if __name__ == "__main__":
os.environ.setdefault("DJANGO_SETTINGS_MODULE", "config.settings.production")
# we're doing this here since otherwise, missing environment
# files in settings result in AttributeError being raised, generating
# a cryptic django.core.exceptions.AppRegistryNotReady error.
# To prevent that, we explicitely load settings here before anything
# else, so we fail fast with a relevant error. See #140 for more details.
django.setup()
from django.core.management import execute_from_command_line

View File

@ -0,0 +1 @@
Better error messages in case of missing environment variables (#140)