{"action":"created","issue":{"url":"https://api.github.com/repos/jimsalterjrs/sanoid/issues/215","repository_url":"https://api.github.com/repos/jimsalterjrs/sanoid","labels_url":"https://api.github.com/repos/jimsalterjrs/sanoid/issues/215/labels{/name}","comments_url":"https://api.github.com/repos/jimsalterjrs/sanoid/issues/215/comments","events_url":"https://api.github.com/repos/jimsalterjrs/sanoid/issues/215/events","html_url":"https://github.com/jimsalterjrs/sanoid/issues/215","id":318967472,"node_id":"MDU6SXNzdWUzMTg5Njc0NzI=","number":215,"title":"Remote hosts don't work unless `@` is included","user":{"login":"candlerb","id":44789,"node_id":"MDQ6VXNlcjQ0Nzg5","avatar_url":"https://avatars2.githubusercontent.com/u/44789?v=4","gravatar_id":"","url":"https://api.github.com/users/candlerb","html_url":"https://github.com/candlerb","followers_url":"https://api.github.com/users/candlerb/followers","following_url":"https://api.github.com/users/candlerb/following{/other_user}","gists_url":"https://api.github.com/users/candlerb/gists{/gist_id}","starred_url":"https://api.github.com/users/candlerb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/candlerb/subscriptions","organizations_url":"https://api.github.com/users/candlerb/orgs","repos_url":"https://api.github.com/users/candlerb/repos","events_url":"https://api.github.com/users/candlerb/events{/privacy}","received_events_url":"https://api.github.com/users/candlerb/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-04-30T17:19:23Z","updated_at":"2018-09-03T16:25:54Z","closed_at":null,"author_association":"NONE","body":"Syncoid usage shows that remote hosts only need a colon in them, and the `user@` part is optional:\r\n\r\n```\r\nUsage:\r\n     syncoid [options]... SOURCE TARGET\r\n     or   syncoid [options]... SOURCE [USER@]HOST:TARGET\r\n     or   syncoid [options]... [USER@]HOST:SOURCE [TARGET]\r\n     or   syncoid [options]... [USER@]HOST:SOURCE [USER@]HOST:TARGET\r\n```\r\n\r\nHowever in practice, it fails unless you supply the `user@` part.  The code enforces this in `sub getssh`:\r\n\r\n```\r\n        # if we got passed something with an @ in it, we assume it's an ssh connection, eg root@myotherbox\r\n        if ($fs =~ /\\@/) {\r\n```\r\n\r\nI suggest either updating the usage docs, or looking for `:` rather than `@` to determine if the target is remote."},"comment":{"url":"https://api.github.com/repos/jimsalterjrs/sanoid/issues/comments/418156650","html_url":"https://github.com/jimsalterjrs/sanoid/issues/215#issuecomment-418156650","issue_url":"https://api.github.com/repos/jimsalterjrs/sanoid/issues/215","id":418156650,"node_id":"MDEyOklzc3VlQ29tbWVudDQxODE1NjY1MA==","user":{"login":"phreaker0","id":12198141,"node_id":"MDQ6VXNlcjEyMTk4MTQx","avatar_url":"https://avatars0.githubusercontent.com/u/12198141?v=4","gravatar_id":"","url":"https://api.github.com/users/phreaker0","html_url":"https://github.com/phreaker0","followers_url":"https://api.github.com/users/phreaker0/followers","following_url":"https://api.github.com/users/phreaker0/following{/other_user}","gists_url":"https://api.github.com/users/phreaker0/gists{/gist_id}","starred_url":"https://api.github.com/users/phreaker0/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phreaker0/subscriptions","organizations_url":"https://api.github.com/users/phreaker0/orgs","repos_url":"https://api.github.com/users/phreaker0/repos","events_url":"https://api.github.com/users/phreaker0/events{/privacy}","received_events_url":"https://api.github.com/users/phreaker0/received_events","type":"User","site_admin":false},"created_at":"2018-09-03T16:25:54Z","updated_at":"2018-09-03T16:25:54Z","author_association":"CONTRIBUTOR","body":"':' is a valid char in zfs dataset names, so it can't be uses properly for detecting remote datasets, so it's best to update the documentation"}}
{"actor":{"display_login":"phreaker0"}}