-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Recent commit to S3BotoStorageMixin forces use of preload_metadata #48
Comments
I'm not across any of this at all unfortunately, but if you have a fix in mind, it's certainly welcome! |
I've recently deployed a Mezzanine project using AWS and got the same problem, |
This cost me about 8 hours by the way, i couldn't for the life of me figure out why this was failing. |
I'm glad this issue could help at least. I'd submit a PR, but I'm no longer using Mezzanine for any active projects. |
The commit 8d91c77 seems to have an unintended consequence. The
entries
dictionary is only populated ifpreload_metadata = True
(https://bitbucket.org/david/django-storages/src/f153a70ba254dc129d9403546809a02256ef75b5/storages/backends/s3boto.py?at=default#s3boto.py-302)If
preload_metadata = False
(which is the default) then this causes aKeyError
when there is anything in the bucket.It seems instead that this should call self.size rather than using
self.entries
directly.self.size(name)
does everything the recent change does but without requiringpreload_metadata = True.
The text was updated successfully, but these errors were encountered: