From f12d4ac1a6833a9764035ef32c841ee5d12e066b Mon Sep 17 00:00:00 2001 From: Chris Lovering Date: Tue, 7 Sep 2021 18:58:33 +0100 Subject: Remove explicit check for db port, as it's optional We also do not specifiy it in prod, so this crash looped when trying to deploy --- manage.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/manage.py b/manage.py index 1e613e93..648d6635 100755 --- a/manage.py +++ b/manage.py @@ -60,7 +60,6 @@ class SiteManager: db_url_parts = urlsplit(db_url) if not all(( db_url_parts.hostname, - db_url_parts.port, db_url_parts.username, db_url_parts.password, db_url_parts.path @@ -107,7 +106,8 @@ class SiteManager: # Get database URL based on environmental variable passed in compose database_url_parts = SiteManager.parse_db_url(os.environ["DATABASE_URL"]) domain = database_url_parts.hostname - port = database_url_parts.port + # Port may be omitted, 5432 is the default psql port + port = database_url_parts.port or 5432 # Attempt to connect to the database socket s = socket.socket(socket.AF_INET, socket.SOCK_STREAM) -- cgit v1.2.3