{"action":"created","issue":{"url":"https://api.github.com/repos/remind101/empire/issues/550","labels_url":"https://api.github.com/repos/remind101/empire/issues/550/labels{/name}","comments_url":"https://api.github.com/repos/remind101/empire/issues/550/comments","events_url":"https://api.github.com/repos/remind101/empire/issues/550/events","html_url":"https://github.com/remind101/empire/issues/550","id":90511367,"number":550,"title":"Proposal: Implement `emp run` within Empire","user":{"login":"ejholmes","id":95166,"avatar_url":"https://avatars.githubusercontent.com/u/95166?v=3","gravatar_id":"","url":"https://api.github.com/users/ejholmes","html_url":"https://github.com/ejholmes","followers_url":"https://api.github.com/users/ejholmes/followers","following_url":"https://api.github.com/users/ejholmes/following{/other_user}","gists_url":"https://api.github.com/users/ejholmes/gists{/gist_id}","starred_url":"https://api.github.com/users/ejholmes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ejholmes/subscriptions","organizations_url":"https://api.github.com/users/ejholmes/orgs","repos_url":"https://api.github.com/users/ejholmes/repos","events_url":"https://api.github.com/users/ejholmes/events{/privacy}","received_events_url":"https://api.github.com/users/ejholmes/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/remind101/empire/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2015-06-23T22:59:10Z","updated_at":"2015-07-01T21:10:59Z","closed_at":"2015-07-01T17:53:18Z","body":"Right now, there's a couple of disadvantages with the separate [relay](https://github.com/remind101/empire/tree/master/relay):\r\n\r\n1. Since it's separate, it's another dependency you have to run alongside Empire, and you need to configure Empire to know where to find the relay.\r\n2. The relay is fairly complicated to setup, since it requires listening on both HTTP and TCP with TLS.\r\n3. It's difficult to horizontally scale it in it's current implementation (you can only have 1 process running atm).\r\n\r\nNow that we've separated out the `emp` CLI, I think it might be worthwhile to break Heroku API compatibility here with something that more closely matches the [attach](https://docs.docker.com/reference/api/docker_remote_api_v1.19/#attach-to-a-container) endpoint of the Docker daemon API. The primary advantages would be:\r\n\r\n1. Nothing extra to run. Spin up Empire and `emp run`'s will just work, keeping with our goal of making Empire simple to run.\r\n2. It has no impact on horizontally scaling Empire, since you'll simply attach to the stdout of the container from the HTTP request instead of having to re-connect to a TCP endpoint.\r\n\r\nI think the only real disadvantage here is that we'll break `hk run`, but I think it would be worth it.\r\n\r\n/cc @phobologic @bmarini "},"comment":{"url":"https://api.github.com/repos/remind101/empire/issues/comments/117825677","html_url":"https://github.com/remind101/empire/issues/550#issuecomment-117825677","issue_url":"https://api.github.com/repos/remind101/empire/issues/550","id":117825677,"user":{"login":"rgabo","id":96628,"avatar_url":"https://avatars.githubusercontent.com/u/96628?v=3","gravatar_id":"","url":"https://api.github.com/users/rgabo","html_url":"https://github.com/rgabo","followers_url":"https://api.github.com/users/rgabo/followers","following_url":"https://api.github.com/users/rgabo/following{/other_user}","gists_url":"https://api.github.com/users/rgabo/gists{/gist_id}","starred_url":"https://api.github.com/users/rgabo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rgabo/subscriptions","organizations_url":"https://api.github.com/users/rgabo/orgs","repos_url":"https://api.github.com/users/rgabo/repos","events_url":"https://api.github.com/users/rgabo/events{/privacy}","received_events_url":"https://api.github.com/users/rgabo/received_events","type":"User","site_admin":false},"created_at":"2015-07-01T21:10:59Z","updated_at":"2015-07-01T21:10:59Z","body":"@ejholmes I did, but the official release took precedence in my PATH :smile: Works great now, thanks."}}