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
Ignore duplicate files when uploading, instead of throwing error #2284
Labels
Comments
Just to be clear, this should only no-op on files with the exact same hash, if someone tries to upload a different file with the same name that should still get the error as it is today. |
dstufft
added
good first issue
This issue is ideal for first-time contributors!
Post launch - low priority
labels
Sep 17, 2017
benjaminws
added a commit
to benjaminws/warehouse
that referenced
this issue
Sep 19, 2017
- pypi#2284 - Move duplication check to happen after the temp file is persisted, so a hash can be generated - Create function to quuery for existing file and check hash - Update unit tests to generate hash from file content, not name since we need to test it - Move file size check to happen after duplication check
dstufft
added a commit
to benjaminws/warehouse
that referenced
this issue
Sep 22, 2017
benjaminws
added a commit
to benjaminws/warehouse
that referenced
this issue
Sep 22, 2017
…jaminws/warehouse into fix/pypigh-2284-duplicate-file-check
dstufft
added a commit
to benjaminws/warehouse
that referenced
this issue
Sep 22, 2017
dstufft
pushed a commit
that referenced
this issue
Sep 22, 2017
Fixed in #2419. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Original complaint: https://gist.github.com/boppreh/ac7522b3a4ac46b4f6010eecddc57f21
And Donald's email reply on Distutils:
The text was updated successfully, but these errors were encountered: