Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

snapctl should do url checking and throw a better error #260

Closed
geauxvirtual opened this issue Jul 7, 2015 · 5 comments
Closed

snapctl should do url checking and throw a better error #260

geauxvirtual opened this issue Jul 7, 2015 · 5 comments

Comments

@geauxvirtual
Copy link
Contributor

snapctl should do url checking and throw a formatting error if a user does not input a url in the format snapctl is expecting.

screen shot 2015-07-07 at 12 09 45 pm

@lynxbat
Copy link
Contributor

lynxbat commented Jul 7, 2015

This goes back to my ask for SNAP_URL to be SNAP_HOST (server:ip)

@lynxbat
Copy link
Contributor

lynxbat commented Jul 7, 2015

We should expect the user to write the URL

@pittma pittma added this to the Beta milestone Jul 9, 2015
@lynxbat lynxbat modified the milestone: Beta Aug 8, 2015
@mbbroberg
Copy link
Contributor

s/pulse/snap/g

@mbbroberg
Copy link
Contributor

Still no good as of current version.

build/bin/snapctl --url asdfasdf task list
Error getting tasks:
Get asdfasdf/v1/tasks: unsupported protocol scheme ""

@mbbroberg mbbroberg changed the title pulsectl should do url checking and throw a better error snapctl should do url checking and throw a better error Feb 23, 2016
@tiffanyfay
Copy link
Contributor

I'll tinker around with this.

tiffanyfay added a commit to tiffanyfay/snap that referenced this issue Feb 24, 2016
tiffanyfay added a commit to tiffanyfay/snap that referenced this issue Feb 24, 2016
tiffanyfay added a commit to tiffanyfay/snap that referenced this issue Feb 24, 2016
tiffanyfay added a commit to tiffanyfay/snap that referenced this issue Feb 29, 2016
tiffanyfay added a commit to tiffanyfay/snap that referenced this issue Mar 1, 2016
tiffanyfay added a commit to tiffanyfay/snap that referenced this issue Mar 1, 2016
tiffanyfay added a commit to tiffanyfay/snap that referenced this issue Mar 1, 2016
tiffanyfay added a commit that referenced this issue Mar 3, 2016
Fixes #260 Add url checking and comment about invalid target
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants