CachedFilesMixin is not compatible with S3BotoStorage.

[idanzalz@…]

CachedFilesMixin breaks S3BotoStorage when used together because it applies "unquote" on the result of the storage url method.
see ​https://github.com/django/django/pull/321 for more details

[Jannis Leidel]

Please don't just point to a pull request, this ticket exists to discuss the issue.

[anonymous]

There isn't really that much to add. The storage's url method result gets wrapped with unquote when using the CachedFilesMixin.
This sometimes produces a wrong url for the S3BotoStorage, since the Signature part in the url may contain %2B which is translated to '+' by unquote and then you get an authentication error from AWS when trying to remove it.

Calling unquote on the url seems like a side effect of the CachedFilesMixin that has no connection to what it's supposed to do so I think it should be removed.

[Aymeric Augustin]

This ticket looks valid. But the current patch isn't acceptable because it breaks tests (as mentioned by Karen on the pull request).

To move forward, the code should make it clearer which kind of path (filesystem, URL-unescaped, URL-escaped) in accepted and return by storage functions, and more tests must be added.

[Mariusz Felisiak]

This issue is not valid anymore because CachedFilesMixin and CachedStaticFilesStorage will be removed in Django 3.1 (see f1894bae3071da4ee577fc40ae61491f3e03d82c) and fix doesn't qualify for a backport.