Handle binary file
A way to handle binary file without any file size limits
Hi!
First of all, thank you for your contribution!
I have a couple of comments:
- I think it is better move
requirements.txttorequirements_test.txtto make it clear that they are only for tests. - Also I suggest that creating temp file must be controlled by a flag passed to
to_curlfunction because writing files are slow and not always needed. For example, when this library is used on remote server to which you don't have a direct access to run curl from its environment but you have access only to its logs collected by another server. So one can't really reproduce request on local machine when file was generated on a remote one.
Hello :)
I think it is better move
requirements.txttorequirements_test.txtto make it clear that they are only for tests.
Indeed, that's way better :)
Also I suggest that creating temp file must be controlled by a flag passed to
to_curlfunction because writing files are slow and not always needed. For example, when this library is used on remote server to which you don't have a direct access to run curl from its environment but you have access only to its logs collected by another server. So one can't really reproduce request on local machine when file was generated on a remote one.
OK, i give a try using a flag for switching between the tempfile method and an inline one (if possible)
OK, i give a try using a flag for switching between the tempfile method and an inline one (if possible)
Let me know please if you are going to add this flag. When it is added I'll merge this request.