Open dustymabe opened 8 years ago
looks like it blindly tacks on http://
to the front of whatever host is provided. We should probably make the code a little smarter.
I don't think this is a bug at all: the doc clearly says:
-H, --host string API hostname (default "almighty.io")
Garbage in-garbage out.
@tsmaeder Doc or no doc, doesn't mean we can't handle it.
@tsmaeder so you have to do -h blah.io -s https
? that seems overly tedious knowing most humans and computers by now like to use urls too :)
@maxandersen but.. the client knows the 'base' of the URL.. scheme:host/base.. we could infer scheme via the malformatted 'host' http://xxx, but do we override 'base' as well? or do we always append base?
Normally, a user wouldn't have to provide --host at all, as it will default point to production This is technically a 'dev' problem.
maybe just add support for --url
?
Seems like this works fine using curl:
but going through alm-cli I get complaints of invalid URL escape: