curl_translate() bugs

[mgirlich]

A couple of bugs I found when creating the PR for curlconverter.com

Crashes

library(httr2)
curl_translate("curl -X GET \"http://localhost:28139/cookies\" -H \"accept: application/json\" -H 'Cookie: mysamplecookie=someValue; emptycookie=; otherCookie=2'")
#> Error in `httr2::url_parse()`:
#> ! `url` must be a single string, not a character vector.

Wrong url

curl_translate("curl httpbin.org")
#> request("/httpbin.org") |> 
#>   req_perform()

Missing headers:

curl_translate("curl --header \"Content-Type: text/xml;charset=UTF-8\" --header \"getWorkOrderCancel;\" localhost:28139/get")
#> request("localhost:///28139/get") |> 
#>   req_perform()

^{Created on 2024-07-12 with reprex v2.1.0}

[hadley]

1. I think maybe we shouldn't bother supporting this, since adjusting our parse to allow the url to be in any position is going to be a lot of work, and I don't think it's that common in the wild.
2. Now that we use curl to parse urls maybe we can get this working: Flexible URL parse? jeroen/curl#368
3. Assuming the last one is a typo and should be --header \"getWorkOrderCancel;\", I'll have a PR shortly.