{"action":"created","issue":{"url":"https://api.github.com/repos/cylc/cylc/issues/1395","labels_url":"https://api.github.com/repos/cylc/cylc/issues/1395/labels{/name}","comments_url":"https://api.github.com/repos/cylc/cylc/issues/1395/comments","events_url":"https://api.github.com/repos/cylc/cylc/issues/1395/events","html_url":"https://github.com/cylc/cylc/pull/1395","id":64639754,"number":1395,"title":"Log user-at-host of command origin.","user":{"login":"hjoliver","id":2362137,"avatar_url":"https://avatars.githubusercontent.com/u/2362137?v=3","gravatar_id":"","url":"https://api.github.com/users/hjoliver","html_url":"https://github.com/hjoliver","followers_url":"https://api.github.com/users/hjoliver/followers","following_url":"https://api.github.com/users/hjoliver/following{/other_user}","gists_url":"https://api.github.com/users/hjoliver/gists{/gist_id}","starred_url":"https://api.github.com/users/hjoliver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjoliver/subscriptions","organizations_url":"https://api.github.com/users/hjoliver/orgs","repos_url":"https://api.github.com/users/hjoliver/repos","events_url":"https://api.github.com/users/hjoliver/events{/privacy}","received_events_url":"https://api.github.com/users/hjoliver/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"matthewrmshin","id":908613,"avatar_url":"https://avatars.githubusercontent.com/u/908613?v=3","gravatar_id":"","url":"https://api.github.com/users/matthewrmshin","html_url":"https://github.com/matthewrmshin","followers_url":"https://api.github.com/users/matthewrmshin/followers","following_url":"https://api.github.com/users/matthewrmshin/following{/other_user}","gists_url":"https://api.github.com/users/matthewrmshin/gists{/gist_id}","starred_url":"https://api.github.com/users/matthewrmshin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthewrmshin/subscriptions","organizations_url":"https://api.github.com/users/matthewrmshin/orgs","repos_url":"https://api.github.com/users/matthewrmshin/repos","events_url":"https://api.github.com/users/matthewrmshin/events{/privacy}","received_events_url":"https://api.github.com/users/matthewrmshin/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/cylc/cylc/milestones/5","html_url":"https://github.com/cylc/cylc/milestones/soon","labels_url":"https://api.github.com/repos/cylc/cylc/milestones/5/labels","id":262614,"number":5,"title":"soon","description":"A few months in the future. Top priority issues and easy fixes should initially be marked against this milestone, and then moved to *next-release* if work progresses quickly enough.","creator":{"login":"hjoliver","id":2362137,"avatar_url":"https://avatars.githubusercontent.com/u/2362137?v=3","gravatar_id":"","url":"https://api.github.com/users/hjoliver","html_url":"https://github.com/hjoliver","followers_url":"https://api.github.com/users/hjoliver/followers","following_url":"https://api.github.com/users/hjoliver/following{/other_user}","gists_url":"https://api.github.com/users/hjoliver/gists{/gist_id}","starred_url":"https://api.github.com/users/hjoliver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjoliver/subscriptions","organizations_url":"https://api.github.com/users/hjoliver/orgs","repos_url":"https://api.github.com/users/hjoliver/repos","events_url":"https://api.github.com/users/hjoliver/events{/privacy}","received_events_url":"https://api.github.com/users/hjoliver/received_events","type":"User","site_admin":false},"open_issues":57,"closed_issues":0,"state":"open","created_at":"2013-02-08T07:07:13Z","updated_at":"2015-04-13T08:19:31Z","due_on":"2015-05-30T23:00:00Z","closed_at":null},"comments":8,"created_at":"2015-03-26T21:36:34Z","updated_at":"2015-04-14T04:19:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cylc/cylc/pulls/1395","html_url":"https://github.com/cylc/cylc/pull/1395","diff_url":"https://github.com/cylc/cylc/pull/1395.diff","patch_url":"https://github.com/cylc/cylc/pull/1395.patch"},"body":"Something else BoM SMS operators like to see: logging of which user issued a command. This is more important for SMS where one server program runs all suites ... but cylc can still have several users sharing a passphrase to the same suite. This change just passes user@host from the client command to the server command (and info) interface, and logs it:\r\n\r\n```\r\n2015-03-27T10:31:38 INFO - Command from oliverh@wrh-1.niwa.co.nz: trigger task(foo,1,False)\r\n2015-03-27T10:31:39 INFO - Command succeeded: trigger task(foo,1,False)\r\n2015-03-27T10:31:40 INFO - [foo.1] -initiate job-submit\r\n```"},"comment":{"url":"https://api.github.com/repos/cylc/cylc/issues/comments/92594459","html_url":"https://github.com/cylc/cylc/pull/1395#issuecomment-92594459","issue_url":"https://api.github.com/repos/cylc/cylc/issues/1395","id":92594459,"user":{"login":"hjoliver","id":2362137,"avatar_url":"https://avatars.githubusercontent.com/u/2362137?v=3","gravatar_id":"","url":"https://api.github.com/users/hjoliver","html_url":"https://github.com/hjoliver","followers_url":"https://api.github.com/users/hjoliver/followers","following_url":"https://api.github.com/users/hjoliver/following{/other_user}","gists_url":"https://api.github.com/users/hjoliver/gists{/gist_id}","starred_url":"https://api.github.com/users/hjoliver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjoliver/subscriptions","organizations_url":"https://api.github.com/users/hjoliver/orgs","repos_url":"https://api.github.com/users/hjoliver/repos","events_url":"https://api.github.com/users/hjoliver/events{/privacy}","received_events_url":"https://api.github.com/users/hjoliver/received_events","type":"User","site_admin":false},"created_at":"2015-04-14T04:19:38Z","updated_at":"2015-04-14T04:19:38Z","body":"I don't think so - why do you ask?  The new broadcast tests pass, and the file is almost identical on this branch and current master:\r\n```\r\n% git diff master -- bin/cylc-broadcast | cat\r\ndiff --git a/bin/cylc-broadcast b/bin/cylc-broadcast\r\nindex ccfde52..0629e9c 100755\r\n--- a/bin/cylc-broadcast\r\n+++ b/bin/cylc-broadcast\r\n@@ -32,7 +32,7 @@ from cylc import cylc_pyro_client\r\n from cylc.command_prep import prep_pyro\r\n from cylc.print_tree import print_tree\r\n from cylc.task_id import TaskID\r\n-from parsec.validate import validate, ValidationError\r\n+from parsec.validate import validate\r\n from cylc.cfgspec.suite import SPEC\r\n import cylc.flags\r\n```\r\n"}}