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

Jump to bottom

Fixed #34466 -- Reallowed setting cursor_factory in DATABASES["options"] on PostgreSQL. #16736

Merged
merged 1 commit into from Apr 7, 2023
Merged

Fixed #34466 -- Reallowed setting cursor_factory in DATABASES["options"] on PostgreSQL. #16736

merged 1 commit into from Apr 7, 2023
+30 −9

Conversation

andersk
Copy link
Contributor

@andersk andersk commented Apr 6, 2023

felixxm
felixxm reviewed Apr 7, 2023
View reviewed changes
Copy link
Member

@felixxm felixxm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andersk Thanks 👍

django/db/backends/postgresql/base.py Outdated Show resolved Hide resolved
@andersk andersk force-pushed the cursor_factory branch 3 times, most recently from 9b46b91 to fda19a0 Compare April 7, 2023 05:31
@felixxm
Copy link
Member

felixxm commented Apr 7, 2023

@andersk Thanks 👍 I added a release note.

@felixxm felixxm changed the title Fixed #34466 -- Respect overridden psycopg cursor_factory. Fixed #34466 -- Reallowed setting cursor_factory in DATABASES["options"] on PostgreSQL. Apr 7, 2023
@felixxm felixxm merged commit 73cbb37 into django:main Apr 7, 2023
26 checks passed
@andersk andersk deleted the cursor_factory branch April 12, 2023 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@felixxm felixxm felixxm left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@andersk @felixxm