-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
trying to upload empty file fails with a NotImplemented error #399
Comments
👍 I've run into this too |
The same issue was present in the node.js version: aws/aws-sdk-js#15 |
I'm trying to reproduce this, but I'm not able to. Can someone share a set of steps?
|
I was consistently getting this error the day I posted in this issue (3 days ago), but now the exact same command and exact same file are uploading just fine! I I have not updated my awscli package or anything. Maybe something was changed in the S3 API to fix this. Anyway, it works for me now. Thanks. |
Ok, I've added a set of integration tests that verify both I'm going to go ahead and close this now since this appears to be working. If anyone is still running into this, feel free to provide a set of repro steps and we'll investigate. |
I'm still able to hit this one. Maybe it's only in Python 2.7.3?
Here's a snippet from the --debug output:
|
Could you try upgrading to 1.2.0? It looks like this issue is fixed in 1.2.0. |
For what it's worth I've been on 1.1.2 this whole time. edit:
|
My bad, I should have tried that before replying. I'm still able to hit it with 1.2.0, using the same commands from my previous comment.
|
Hmm, it looks like the only difference is that you're using 2.7.3. I'll give that a try and see if I can repro this. |
Tried it on 2.7.3, still not able to repro. Tried on mac/linux and not able to reproduce. Could you provide the part of the debug log that shows the params being sent? It should look like this:
I'm wondering if some extra header is being added to the parameters, which might cause this issue. |
Thanks for looking into this further jamesls. Here's some more of the debug output, starting with that params line:
|
I was still on botocore==0.19.0 After a |
I was trying to sync a directory to s3 with an empty file, and got this error:
A header you provided implies functionality that is not implemented
the command I used:
The text was updated successfully, but these errors were encountered: