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
As well as --data-raw, add --data-binary #35
Comments
@malcolmsparks Sorry for this inconvenience. I changed this in commit b3bf0ba. All the tests still pass and I think this is the saner option to use. Just for my info: do you use this from babashka? Is this blocking for you and do you need a new release? |
@borkdude ah, in that case, I'll close this issue - it's just what I'm looking for. Yes, I'm using this from babashka (via the Arch AUR package). I'm not blocked, I could work around this with a direct call out to |
This is a new babashka binary with the change: https://15545-201467090-gh.circle-artifacts.com/0/release/babashka-0.2.13-SNAPSHOT-linux-amd64.zip Maybe you could test it out. |
Note to self: I wonder what the default should be for posting strings. Now I changed this from |
|
I've tested the new binary on my system (I'll keep it in /usr/bin/bb and report any issues I find). For my file upload tests, it works perfectly, and the files are no longer 'corrupted'. Thanks very much! |
Actually |
@malcolmsparks babashka v0.2.13 released with this bugfix |
According to the curl man page,
Currently, the
--data-raw
option does not work withjava.io.File
values. In my case, I'm trying to upload a binary file to the server, and want to avoid-d
being generated in the curl command because it seems to corrupt files (e.g. CRs and newlines are stripped out).The text was updated successfully, but these errors were encountered: