diff options
-rw-r--r-- | Makefile | 4 | ||||
-rw-r--r-- | core/config.py | 21 | ||||
-rw-r--r-- | core/context.py | 4 | ||||
-rw-r--r-- | docker/Dockerfile (renamed from Dockerfile) | 9 | ||||
-rw-r--r-- | docker/docker-compose.yml (renamed from docker-compose.yml) | 2 | ||||
-rw-r--r-- | docker/start.sh (renamed from scripts/start.sh) | 0 | ||||
-rwxr-xr-x | manage.py | 2 | ||||
-rw-r--r-- | requirements.txt | 1 | ||||
-rw-r--r-- | setup.cfg | 2 | ||||
-rw-r--r-- | takahe/asgi.py | 2 | ||||
-rw-r--r-- | takahe/settings/__init__.py | 0 | ||||
-rw-r--r-- | takahe/settings/base.py (renamed from takahe/settings.py) | 20 | ||||
-rw-r--r-- | takahe/settings/development.py | 13 | ||||
-rw-r--r-- | takahe/settings/production.py | 17 | ||||
-rw-r--r-- | takahe/settings/testing.py | 4 | ||||
-rw-r--r-- | takahe/wsgi.py | 2 | ||||
-rw-r--r-- | users/views/identity.py | 4 |
17 files changed, 75 insertions, 32 deletions
diff --git a/Makefile b/Makefile new file mode 100644 index 0000000..b87b2ce --- /dev/null +++ b/Makefile @@ -0,0 +1,4 @@ +.PHONY: clean + +image: + docker build -t takahe -f docker/Dockerfile . diff --git a/core/config.py b/core/config.py index 0f09404..b9f6878 100644 --- a/core/config.py +++ b/core/config.py @@ -1,3 +1,20 @@ -class Config: +import pydantic - pass + +class Config(pydantic.BaseModel): + + # Basic configuration options + site_name: str = "takahē" + identity_max_age: int = 24 * 60 * 60 + + # Cached ORM object storage + __singleton__ = None + + class Config: + env_prefix = "takahe_" + + @classmethod + def load(cls) -> "Config": + if cls.__singleton__ is None: + cls.__singleton__ = cls() + return cls.__singleton__ diff --git a/core/context.py b/core/context.py index 026ac11..17617b9 100644 --- a/core/context.py +++ b/core/context.py @@ -1,7 +1,7 @@ -from django.conf import settings +from core.config import Config def config_context(request): return { - "config": {"site_name": settings.SITE_NAME}, + "config": Config.load(), } diff --git a/Dockerfile b/docker/Dockerfile index 1f62240..14e033b 100644 --- a/Dockerfile +++ b/docker/Dockerfile @@ -1,4 +1,6 @@ -FROM python:3.9-bullseye as builder +# Build stage + +FROM python:3.11.0-buster as builder RUN mkdir -p /takahe RUN python -m venv /takahe/.venv @@ -12,8 +14,9 @@ RUN . /takahe/.venv/bin/activate \ && pip install --upgrade pip \ && pip install --upgrade -r requirements.txt +# Final image stage -FROM python:3.9-slim-bullseye +FROM python:3.11.0-slim-buster RUN apt-get update && apt-get install -y libpq5 @@ -23,4 +26,4 @@ COPY . /takahe WORKDIR /takahe EXPOSE 8000 -CMD ["/takahe/scripts/start.sh"] +CMD ["/takahe/docker/start.sh"] diff --git a/docker-compose.yml b/docker/docker-compose.yml index f64bfb6..00463bf 100644 --- a/docker-compose.yml +++ b/docker/docker-compose.yml @@ -19,7 +19,7 @@ services: build: . image: tahake:latest environment: - - "DJANGO_SETTINGS_MODULE=takahe.settings" + - "DJANGO_SETTINGS_MODULE=takahe.settings.production" - "SECRET_KEY=insecure_secret" - "POSTGRES_HOST=db" - "POSTGRES_DB=tahake" diff --git a/scripts/start.sh b/docker/start.sh index 99f1ed0..99f1ed0 100644 --- a/scripts/start.sh +++ b/docker/start.sh @@ -6,7 +6,7 @@ import sys def main(): """Run administrative tasks.""" - os.environ.setdefault("DJANGO_SETTINGS_MODULE", "takahe.settings") + os.environ.setdefault("DJANGO_SETTINGS_MODULE", "takahe.settings.production") try: from django.core.management import execute_from_command_line except ImportError as exc: diff --git a/requirements.txt b/requirements.txt index dbe6fb0..e897cfc 100644 --- a/requirements.txt +++ b/requirements.txt @@ -12,3 +12,4 @@ psycopg2~=2.9.5 bleach~=5.0.1 pytest-django~=4.5.2 pytest-httpx~=0.21 +pydantic~=1.10.2 @@ -9,7 +9,7 @@ multi_line_output = 3 [tool:pytest] addopts = --tb=short -DJANGO_SETTINGS_MODULE = takahe.settings +DJANGO_SETTINGS_MODULE = takahe.settings.testing filterwarnings = ignore:There is no current event loop diff --git a/takahe/asgi.py b/takahe/asgi.py index 99a9cfb..3424b23 100644 --- a/takahe/asgi.py +++ b/takahe/asgi.py @@ -11,6 +11,6 @@ import os from django.core.asgi import get_asgi_application -os.environ.setdefault("DJANGO_SETTINGS_MODULE", "takahe.settings") +os.environ.setdefault("DJANGO_SETTINGS_MODULE", "takahe.settings.production") application = get_asgi_application() diff --git a/takahe/settings/__init__.py b/takahe/settings/__init__.py new file mode 100644 index 0000000..e69de29 --- /dev/null +++ b/takahe/settings/__init__.py diff --git a/takahe/settings.py b/takahe/settings/base.py index e8982ae..a2ccb98 100644 --- a/takahe/settings.py +++ b/takahe/settings/base.py @@ -1,17 +1,7 @@ import os from pathlib import Path -# Build paths inside the project like this: BASE_DIR / 'subdir'. -BASE_DIR = Path(__file__).resolve().parent.parent - -# SECURITY WARNING: keep the secret key used in production secret! -SECRET_KEY = os.environ.get("SECRET_KEY", "insecure_secret") - -# SECURITY WARNING: don't run with debug turned on in production! -DEBUG = True - -ALLOWED_HOSTS = ["*"] -CSRF_TRUSTED_ORIGINS = ["http://*", "https://*"] +BASE_DIR = Path(__file__).resolve().parent.parent.parent # Application definition @@ -30,7 +20,6 @@ INSTALLED_APPS = [ ] MIDDLEWARE = [ - "core.middleware.AlwaysSecureMiddleware", "django.middleware.security.SecurityMiddleware", "django.contrib.sessions.middleware.SessionMiddleware", "django.middleware.common.CommonMiddleware", @@ -115,9 +104,4 @@ STATICFILES_DIRS = [ BASE_DIR / "static", ] -CRISPY_FAIL_SILENTLY = not DEBUG - -SITE_NAME = "takahē" -DEFAULT_DOMAIN = "feditest.aeracode.org" -ALLOWED_DOMAINS = ["feditest.aeracode.org"] -IDENTITY_MAX_AGE = 24 * 60 * 60 +ALLOWED_HOSTS = ["*"] diff --git a/takahe/settings/development.py b/takahe/settings/development.py new file mode 100644 index 0000000..4e0098b --- /dev/null +++ b/takahe/settings/development.py @@ -0,0 +1,13 @@ +import os + +from .base import * # noqa + +# Load secret key from environment with a fallback +SECRET_KEY = os.environ.get("TAKAHE_SECRET_KEY", "insecure_secret") + +# Disable the CRSF origin protection +MIDDLEWARE.insert(0, "core.middleware.AlwaysSecureMiddleware") + +# Ensure debug features are on +DEBUG = True +CRISPY_FAIL_SILENTLY = False diff --git a/takahe/settings/production.py b/takahe/settings/production.py new file mode 100644 index 0000000..2f943f4 --- /dev/null +++ b/takahe/settings/production.py @@ -0,0 +1,17 @@ +import os + +from .base import * # noqa + +# Load secret key from environment +try: + SECRET_KEY = os.environ["TAKAHE_SECRET_KEY"] +except KeyError: + print("You must specify the TAKAHE_SECRET_KEY environment variable!") + os._exit(1) + +# Ensure debug features are off +DEBUG = False +CRISPY_FAIL_SILENTLY = True + +# TODO: Allow better setting of allowed_hosts, if we need to +ALLOWED_HOSTS = ["*"] diff --git a/takahe/settings/testing.py b/takahe/settings/testing.py new file mode 100644 index 0000000..6527333 --- /dev/null +++ b/takahe/settings/testing.py @@ -0,0 +1,4 @@ +from .base import * # noqa + +# Fixed secret key +SECRET_KEY = "testing_secret" diff --git a/takahe/wsgi.py b/takahe/wsgi.py index 05ae06f..c8ad0a0 100644 --- a/takahe/wsgi.py +++ b/takahe/wsgi.py @@ -11,6 +11,6 @@ import os from django.core.wsgi import get_wsgi_application -os.environ.setdefault("DJANGO_SETTINGS_MODULE", "takahe.settings") +os.environ.setdefault("DJANGO_SETTINGS_MODULE", "takahe.settings.production") application = get_wsgi_application() diff --git a/users/views/identity.py b/users/views/identity.py index 5d11d63..d78bda1 100644 --- a/users/views/identity.py +++ b/users/views/identity.py @@ -1,13 +1,13 @@ import string from django import forms -from django.conf import settings from django.contrib.auth.decorators import login_required from django.http import Http404 from django.shortcuts import redirect from django.utils.decorators import method_decorator from django.views.generic import FormView, TemplateView, View +from core.config import Config from core.forms import FormHelper from users.decorators import identity_required from users.models import Domain, Follow, Identity, IdentityStates @@ -26,7 +26,7 @@ class ViewIdentity(TemplateView): fetch=True, ) posts = identity.posts.all()[:100] - if identity.data_age > settings.IDENTITY_MAX_AGE: + if identity.data_age > Config.load().IDENTITY_MAX_AGE: identity.transition_perform(IdentityStates.outdated) return { "identity": identity, |