Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DBTP-457 Use dbt-copilot-python to set ALLOWED_HOSTS & DATABASES in DBT PaaS #461

Merged
merged 7 commits into from
Sep 22, 2023
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ npm run build
make first-use # ... use make up after this has run
```

\* You will need SSO auth details to allow the project to run, but there are a lot of other details that would help make the experience better; ask another dev for their .env so that you can get a headstart
\* You will need SSO auth details (`AUTHBROKER_*` in your `.env` file) to allow the project to run, but there are a lot of other details that would help make the experience better; ask another dev for their .env so that you can get a head start.

## Run front end in watch mode

Expand Down
10 changes: 7 additions & 3 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 4 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,10 @@
"clean": "rm -f ./assets/webpack_bundles/*"
},
"browserslist": [
"last 3 chrome versions", "last 1 firefox version", "last 1 safari version", "IE 11"
"last 3 chrome versions",
"last 1 firefox version",
"last 1 safari version",
"IE 11"
CamLamb marked this conversation as resolved.
Show resolved Hide resolved
],
"engines": {
"npm": ">8.0.0",
Expand Down
20 changes: 19 additions & 1 deletion poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ boto3 = "^1.26.115"
pyjwt = { extras = ["crypto"], version = "^2.6.0" }
django-taggit = "^3.1.0"
django-celery-beat = "^2.5.0"
dbt-copilot-python = "^0.1.3"

[tool.poetry.group.dev.dependencies]
poetry = "^1.4.2"
Expand Down
1 change: 1 addition & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ colorama==0.4.6 ; python_version >= "3.11" and python_version < "4.0" and platfo
crispy-forms-gds==0.2.4 ; python_version >= "3.11" and python_version < "4.0"
cron-descriptor==1.4.0 ; python_version >= "3.11" and python_version < "4.0"
cryptography==41.0.3 ; python_version >= "3.11" and python_version < "4.0"
dbt-copilot-python==0.1.2 ; python_version >= "3.11" and python_version < "4.0"
defusedxml==0.7.1 ; python_version >= "3.11" and python_version < "4.0"
django-audit-log-middleware==0.0.4 ; python_version >= "3.11" and python_version < "4.0"
django-celery-beat==2.5.0 ; python_version >= "3.11" and python_version < "4.0"
Expand Down
3 changes: 2 additions & 1 deletion src/config/settings/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@

import environ
import sentry_sdk
from dbt_copilot_python.network import setup_allowed_hosts
from django.urls import reverse_lazy
from sentry_sdk.integrations.django import DjangoIntegration
from sentry_sdk.integrations.redis import RedisIntegration
Expand Down Expand Up @@ -89,7 +90,7 @@

# Allow all hosts
# (this application will always be run behind a PaaS router or locally)
ALLOWED_HOSTS = ["*"]
ALLOWED_HOSTS = setup_allowed_hosts(["*"])
CamLamb marked this conversation as resolved.
Show resolved Hide resolved

# Set up Django
LOCAL_APPS = [
Expand Down