#26177 closed Bug (fixed)
Using a Postgres database with TIME_ZONE set to None and USE_TZ set to False causes an exception
Reported by: | Jensen Cochran | Owned by: | Tim Graham |
---|---|---|---|
Component: | Database layer (models, ORM) | Version: | 1.8 |
Severity: | Release blocker | Keywords: | postgres postgresql time zone time_zone TIME_ZONE=None |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
You are supposed to be able to set TIME_ZONE = None and USE_TZ = False in order to the use the system time (if you are not on Windows). When using a Postgres database, this functionality is broken in 1.8 onward and causes an exception to occur when the development server is started.
The bottom part of the stack trace looks like this:
File "/usr/lib64/python2.7/site-packages/django/db/backends/base/base.py", line 121, in connect self.init_connection_state() File "/usr/lib64/python2.7/site-packages/django/db/backends/postgresql_psycopg2/base.py", line 204, in init_connection_state cursor.execute(self.ops.set_time_zone_sql(), [tz]) django.db.utils.ProgrammingError: syntax error at or near "NULL"
What is happening is that self.init_connection_state() is not checking whether or not the tz setting is None, so None is being formatted into the SQL string inside of self.ops.set_time_zone_sql() to get "SET TIME ZONE NULL".
Before 1.8, there was a check in place to make sure that tz was truthy before proceeding, but it was removed in this commit.
You can reproduce this bug simply by making a new Django project and setting TIME_ZONE = None, USE_TZ = False, and setting the default database to a postgres database.
Tested on Django 1.8.0, 1.8.9, and 1.9.2 using PostgreSQL 9.2.9 and Python 2.7.3 on CentOS 6.5
Change History (7)
comment:1 by , 9 years ago
Severity: | Normal → Release blocker |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:2 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:3 by , 9 years ago
Has patch: | set |
---|
comment:4 by , 9 years ago
Triage Stage: | Accepted → Ready for checkin |
---|
PR