AbstractAdapter wasn't delegating `binmode` or `binmode?` to the underlying @tempfile object. `binmode` is useful for cases where the Adapter object gets handed on to another library expecting an `IO` object holding image data. We ran into this with Prawn and people using `prawnto` in conjunction with Paperclip. Prawn wants to ensure the `IO` is in binmode for embedding the image data.
…th. Remove illegal filename characters check since it is no longer relevant when doing a hexdigest on the filename
When using S3 for storage, their data consistency model makes the current behaviour of Attachment#reprocess! unreliable. Apparently, Amazon can not guarantee that PUT's called after DELETE's is actually executed in that order. Hence, I propose preserving files, when doing a reprocces.
This ensures that multiline strings are properly matched in their entirety.