> [minor] should be able to use a File payload here? Or do you need to set the 
> length for some reason (I think jclouds should do that internally too, 
> though?).

At this point, it should be the equivalent, but the old code treated `File` and 
`InputStream` differently (`File` worked, and `InputStream` did not), so I 
thought it warranted a test to demonstrate that it now does.

As for the length, `InputStreamPayload` does not get its content length set 
automatically (for non-reversible streams, there is no way to obtain it without 
reading through the stream).

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/192#issuecomment-30198114

Reply via email to