Opened 2 years ago

Closed 2 years ago

#34110 closed New feature (fixed)

Add InMemoryStorage

Reported by: Josh Thomas Owned by: Francesco Panico
Component: File uploads/storage Version: dev
Severity: Normal Keywords:
Cc: Paolo Melchiorre, Francesco Panico 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

I'd like to see Django offer an in memory storage backend, similar to the ​dj-inmemorystorage third-party package.

I have found this package useful in speeding up tests that deal with file storage, but it appears to be unmaintained.

Change History (18)

comment:1 by Paolo Melchiorre, 2 years ago

I agree with the proposal and I have sent an email to the django developer list for feedback

​https://groups.google.com/g/django-developers/c/KzwdJCxzgu4/m/qUROzQBlCQAJ

comment:2 by Paolo Melchiorre, 2 years ago

Cc: Paolo Melchiorre added

comment:3 by Mariusz Felisiak, 2 years ago

Triage Stage: Unreviewed → Accepted

Tentatively accepted, based on the feedback from the mailing list.

comment:4 by Shai Berger, 2 years ago

Consider also #23251 where the issue of storage in tests is addressed more comprehensively

comment:5 by Francesco Panico, 2 years ago

Owner: changed from nobody to Francesco Panico
Status: new → assigned

comment:6 by Francesco Panico, 2 years ago

Cc: Francesco Panico added
Has patch: set

​PR

comment:7 by Mariusz Felisiak, 2 years ago

Needs documentation: set
Patch needs improvement: set

comment:8 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

In 032c09c4:

Refs #34110 -- Reorganized django.core.files.storage into a separate module.

comment:9 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

In 99b4f90e:

Refs #34110 -- Added StorageSettingsMixin.

comment:10 by Francesco Panico, 2 years ago

Needs documentation: unset
Patch needs improvement: unset

comment:11 by Mariusz Felisiak, 2 years ago

Needs tests: set
Patch needs improvement: set

comment:12 by Francesco Panico, 2 years ago

Needs tests: unset
Patch needs improvement: unset

comment:13 by Mariusz Felisiak, 2 years ago

Needs tests: set

comment:14 by Francesco Panico, 2 years ago

Needs tests: unset

comment:15 by Mariusz Felisiak, 2 years ago

Patch needs improvement: set

comment:16 by Francesco Panico, 2 years ago

Patch needs improvement: unset

comment:17 by Mariusz Felisiak, 2 years ago

Triage Stage: Accepted → Ready for checkin

comment:18 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

Resolution: → fixed
Status: assigned → closed

In 72efd840:

Fixed #34110 -- Added in-memory file storage.

Thanks Paolo Melchiorre, Carlton Gibson, and Mariusz Felisiak for
reviews.

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