Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#32687 closed Bug (fixed)

database client runshell doesn't respect os.environ values in some cases

Reported by: Konstantin Alekseev Owned by: Konstantin Alekseev
Component: Core (Management commands) Version: 3.2
Severity: Release blocker Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description (last modified by Konstantin Alekseev)

postgresql client returns empty dict instead of None for env
as a result os.environ is not used and empty env passed
to subprocess.
Bug introduced in https://github.com/django/django/commit/bbe6fbb8768e8fb1aecb96d51c049d7ceaf802d3#diff-e98866ed4d445fbc94bb60bedffd5d8cf07af55dca6e8ffa4945931486efc3eeR23-R26
PR https://github.com/django/django/pull/14315

Change History (7)

comment:1 by Konstantin Alekseev, 3 years ago

Description: modified (diff)

comment:2 by Nick Pope, 3 years ago

Easy pickings: set
Needs tests: set
Severity: NormalRelease blocker
Triage Stage: UnreviewedSomeday/Maybe

Agreed, this is a regression in bbe6fbb8768e8fb1aecb96d51c049d7ceaf802d3.

Please can you add a test and a release note for 3.2.1?

PR

comment:3 by Nick Pope, 3 years ago

Triage Stage: Someday/MaybeAccepted

(Oops, my mistake.)

comment:4 by Nick Pope, 3 years ago

Needs tests: unset
Patch needs improvement: set

comment:5 by Mariusz Felisiak, 3 years ago

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

comment:6 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

Resolution: fixed
Status: assignedclosed

In 6e742dab:

Fixed #32687 -- Restored passing process’ environment to underlying tool in dbshell on PostgreSQL.

Regression in bbe6fbb8768e8fb1aecb96d51c049d7ceaf802d3.

comment:7 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

In 55cb3c8a:

[3.2.x] Fixed #32687 -- Restored passing process’ environment to underlying tool in dbshell on PostgreSQL.

Regression in bbe6fbb8768e8fb1aecb96d51c049d7ceaf802d3.

Backport of 6e742dabc95b00ba896434293556adeb4dbaee8a from main.

Note: See TracTickets for help on using tickets.
Back to Top