{"action":"opened","issue":{"url":"https://api.github.com/repos/xaptum/xtt/issues/81","repository_url":"https://api.github.com/repos/xaptum/xtt","labels_url":"https://api.github.com/repos/xaptum/xtt/issues/81/labels{/name}","comments_url":"https://api.github.com/repos/xaptum/xtt/issues/81/comments","events_url":"https://api.github.com/repos/xaptum/xtt/issues/81/events","html_url":"https://github.com/xaptum/xtt/issues/81","id":376967379,"node_id":"MDU6SXNzdWUzNzY5NjczNzk=","number":81,"title":"Fix inconsistencies in tool help messages and defaults","user":{"login":"zanebeckwith","id":4698352,"node_id":"MDQ6VXNlcjQ2OTgzNTI=","avatar_url":"https://avatars2.githubusercontent.com/u/4698352?v=4","gravatar_id":"","url":"https://api.github.com/users/zanebeckwith","html_url":"https://github.com/zanebeckwith","followers_url":"https://api.github.com/users/zanebeckwith/followers","following_url":"https://api.github.com/users/zanebeckwith/following{/other_user}","gists_url":"https://api.github.com/users/zanebeckwith/gists{/gist_id}","starred_url":"https://api.github.com/users/zanebeckwith/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zanebeckwith/subscriptions","organizations_url":"https://api.github.com/users/zanebeckwith/orgs","repos_url":"https://api.github.com/users/zanebeckwith/repos","events_url":"https://api.github.com/users/zanebeckwith/events{/privacy}","received_events_url":"https://api.github.com/users/zanebeckwith/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-11-02T21:04:58Z","updated_at":"2018-11-02T21:04:58Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"The tool currently has some minor inconsistencies:\r\n- The default for the serverhost parameter to runclient isn't specified, though `localhost` is the default\r\n- The default for the requestid parameter to runclient is incorrect, I think. The default is to *not* request a specific ID (i.e. to use the null ID type), right?\r\n- The default assignedid output name should have a `.txt` extension, not `.bin`, since it's a textual encoding of the ID\r\n- The help message for the rcert parameter to genrootcert is incorrect. It says this is the root's public key output location, but it's the root's certificate output location\r\n- The help messages for the rpriv and rpub parameters to genrootcert are incorrect. They say these are output locations, but they're input locations\r\n- The help message for the serverpub parameter to genservercert is incorrect. It says this is the private key output location, but it's the public key input location.\r\n- The genservercert parameter sid should probably generate a random ID, like genrootcert does, rather than requiring the server_id.bin file. This is currently the only pre-existing file (other than DAA-related things) that a user would have to create themself, so it would be nice to make a sane default like this so everything could be generated anew."}}
{"actor":{"display_login":"zanebeckwith"}}