id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
21602	FileSystemStorage._save() Should Save to a Temporary Filename and Rename to Attempt to be Atomic	Kevin Stone	nobody	"When using FileSystemStorage based backend with say, staticfiles, creates a race condition where it's writing the intended destination file which can lead to race conditions where your web server can serve incomplete assets while they're being written.

Another case happens with post-processing since CachedStaticFilesStorage actually deletes the hash-named file before saving over it.

These issue should be circumventing by directing the writing to a temporary file name and then atomically renaming it to the intended destination.  Even better, FileSystemStorage._save() has to site in a While-loop because of race conditions in get_available_name().  Instead, it could defer calling get_available_name until it's prepared to rename the file to its destination.

I'm working on implementing these changes in my own sub-class of FileSystemStorage (and CachedStaticFilesStorage to avoid the delete() call) and would be apply to submit them as a patch if there's interest."	Cleanup/optimization	new	File uploads/storage	dev	Normal			bcail	Accepted	0	0	0	0	0	0