Opened 10 years ago

Closed 10 years ago

#23600 closed Bug (fixed)

Overriding several settings is ignored by default_storage.

Reported by: Duncan Parkes Owned by: Duncan Parkes
Component: File uploads/storage Version: dev
Severity: Normal Keywords: testing override_settings
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

django.core.files.storage.default_storage keeps a local copy of, and doesn't take account of changes to, the following settings:

MEDIA_URL
FILE_UPLOAD_PERMISSIONS
FILE_UPLOAD_DIRECTORY_PERMISSIONS

This is the problem noted and fixed in #17744 for MEDIA_ROOT and noted in general in #17787.

Change History (6)

comment:1 by Duncan Parkes, 10 years ago

Owner: changed from nobody to Duncan Parkes
Status: new → assigned

comment:2 by Duncan Parkes, 10 years ago

I have a branch ready to pull request for this, but GitHub is misbehaving for me right now.

comment:4 by Collin Anderson, 10 years ago

Triage Stage: Unreviewed → Accepted

comment:5 by Tim Graham, 10 years ago

Patch needs improvement: set

Left comments for improvement on the PR.

comment:6 by Tim Graham <timograham@…>, 10 years ago

Resolution: → fixed
Status: assigned → closed

In 88b6cf4ae41a983bda735a5f61f69f6b1482e2ca:

Fixed #23600 -- Made default_storage aware of more settings changes.

Added MEDIA_URL, FILE_UPLOAD_PERMISSIONS, and
FILE_UPLOAD_DIRECTORY_PERMISSIONS to the list of settings.

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