Skip to content
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

Ensure that we're reading the file from the start #373

Merged
merged 1 commit into from
Aug 1, 2017

Conversation

mr-bo-jangles
Copy link
Contributor

This is needed for Django 1.11 as we can no longer assume that the file we're looking is starting from the beginning, this can cause issues with empty files being saved instead of GZ variants.

@codecov-io
Copy link

codecov-io commented Jul 31, 2017

Codecov Report

Merging #373 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #373      +/-   ##
==========================================
+ Coverage    75.8%   75.82%   +0.01%     
==========================================
  Files          11       11              
  Lines        1550     1551       +1     
==========================================
+ Hits         1175     1176       +1     
  Misses        375      375
Impacted Files Coverage Δ
storages/backends/s3boto3.py 86.54% <100%> (+0.04%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 100c9b7...437a23a. Read the comment docs.

@jschneier
Copy link
Owner

I've just merged a failing test case for this issue, can you rebase to confirm that this then fixes that and then I would be very happy to merge.

@jschneier
Copy link
Owner

Verified locally, thanks.

@jschneier jschneier merged commit b965fb5 into jschneier:master Aug 1, 2017
nitely pushed a commit to satellogic/django-storages that referenced this pull request Jul 30, 2018
nitely pushed a commit to satellogic/django-storages that referenced this pull request Jul 30, 2018
nitely pushed a commit to satellogic/django-storages that referenced this pull request Jul 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants