Opened 6 years ago

Closed 4 years ago

#15641 closed New feature (wontfix)

Allow FileSystemStorage(base_url=None) to disable url access

Reported by: Harm Geerts <hgeerts@…> Owned by: nobody
Component: File uploads/storage Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: yes
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

changeset:9946 changed the behaviour of passing base_url=None to the FileSystemStorage contructor.
Before the change it would disable url access whereas now it enables access through MEDIA_URL

The attached patch restores the old behaviour.

Attachments (2)

15641.diff (2.7 KB) - added by Harm Geerts <hgeerts@…> 6 years ago.
15641-2.diff (3.7 KB) - added by Harm Geerts <hgeerts@…> 6 years ago.
same patch with additional tests for MEDIA_URL as default

Download all attachments as: .zip

Change History (8)

Changed 6 years ago by Harm Geerts <hgeerts@…>

Attachment: 15641.diff added

Changed 6 years ago by Harm Geerts <hgeerts@…>

Attachment: 15641-2.diff added

same patch with additional tests for MEDIA_URL as default

comment:1 Changed 6 years ago by Luke Plant

Type: New feature

comment:2 Changed 6 years ago by Luke Plant

Severity: Normal

comment:3 Changed 6 years ago by Julien Phalip

Needs documentation: set
Triage Stage: UnreviewedDesign decision needed

comment:4 Changed 5 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:5 Changed 5 years ago by Aymeric Augustin

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:6 Changed 4 years ago by Florian Apolloner

Resolution: wontfix
Status: newclosed

Django usually does not and should not serve media files, hence this is imo a non-issue.

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