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.

Sign up for GitHub

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

log warning if DATABASES cannot be defined #2346

Merged
merged 1 commit into from
Feb 22, 2022
+3 −2
Merged

log warning if DATABASES cannot be defined #2346

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
raise warning if error reading from db.conf
@stveit
stveit committed Feb 18, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
commit b9b44e51609cdf1dd8d970601156c4fb928f0b11
5 changes: 3 additions & 2 deletions python/nav/django/settings.py
View file
Original file line number Diff line number Diff line change
@@ -19,6 +19,7 @@
import os
import sys
import copy
import warnings

import django
from django.utils.log import DEFAULT_LOGGING
@@ -66,8 +67,8 @@
},
}
}
except (IOError, OSError):
pass
except (IOError, OSError) as e:
warnings.warn(f"Could not get connection parameters from db.conf: {e}")

# URLs configuration
ROOT_URLCONF = 'nav.django.urls'