[wtn]

Fixes #23, fixes #110.

FilePart sized an IO without #length with File.size(io.local_path). That raised ENOENT ... local.path for IOs with #size but no path, such as Down::ChunkedIO (#110), and NoMethodError for objects passed without UploadIO, such as Rails' ActionDispatch::Http::UploadedFile. It also measured the wrong file when the path no longer led to the open one: deleted after opening, relative after a Dir.chdir, or Tempfile.create(anonymous: true).

FilePart now uses #length, then #size, then the file at local_path, and raises ArgumentError when none gives a length, e.g. a Down::ChunkedIO without a Content-Length.

UploadIO now opens a Pathname as a filename. Pathname responds to #read, but Pathname#read(length) always reads from the start of the file, so with #size in use its uploads would never end.

For Down users: with a gzip-encoded response, ChunkedIO#size is the compressed length. Passing headers: { "Accept-Encoding" => "identity" } to Down.open avoids this.

Types of Changes

• Bug fix.

Contribution

• I added tests for my changes.
• I tested my changes locally.
• I agree to the Developer's Certificate of Origin 1.1.