{"action":"created","issue":{"url":"https://api.github.com/repos/derekparker/delve/issues/88","labels_url":"https://api.github.com/repos/derekparker/delve/issues/88/labels{/name}","comments_url":"https://api.github.com/repos/derekparker/delve/issues/88/comments","events_url":"https://api.github.com/repos/derekparker/delve/issues/88/events","html_url":"https://github.com/derekparker/delve/pull/88","id":63323897,"number":88,"title":"WIP: Refactor to HTTP client/server model","user":{"login":"ironcladlou","id":298299,"avatar_url":"https://avatars.githubusercontent.com/u/298299?v=3","gravatar_id":"","url":"https://api.github.com/users/ironcladlou","html_url":"https://github.com/ironcladlou","followers_url":"https://api.github.com/users/ironcladlou/followers","following_url":"https://api.github.com/users/ironcladlou/following{/other_user}","gists_url":"https://api.github.com/users/ironcladlou/gists{/gist_id}","starred_url":"https://api.github.com/users/ironcladlou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ironcladlou/subscriptions","organizations_url":"https://api.github.com/users/ironcladlou/orgs","repos_url":"https://api.github.com/users/ironcladlou/repos","events_url":"https://api.github.com/users/ironcladlou/events{/privacy}","received_events_url":"https://api.github.com/users/ironcladlou/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/derekparker/delve/milestones/1","html_url":"https://github.com/derekparker/delve/milestones/1.0","labels_url":"https://api.github.com/repos/derekparker/delve/milestones/1/labels","id":967297,"number":1,"title":"1.0","description":"","creator":{"login":"derekparker","id":1314079,"avatar_url":"https://avatars.githubusercontent.com/u/1314079?v=3","gravatar_id":"","url":"https://api.github.com/users/derekparker","html_url":"https://github.com/derekparker","followers_url":"https://api.github.com/users/derekparker/followers","following_url":"https://api.github.com/users/derekparker/following{/other_user}","gists_url":"https://api.github.com/users/derekparker/gists{/gist_id}","starred_url":"https://api.github.com/users/derekparker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/derekparker/subscriptions","organizations_url":"https://api.github.com/users/derekparker/orgs","repos_url":"https://api.github.com/users/derekparker/repos","events_url":"https://api.github.com/users/derekparker/events{/privacy}","received_events_url":"https://api.github.com/users/derekparker/received_events","type":"User","site_admin":false},"open_issues":17,"closed_issues":4,"state":"open","created_at":"2015-02-05T16:06:41Z","updated_at":"2015-04-09T14:00:57Z","due_on":null,"closed_at":null},"comments":88,"created_at":"2015-03-20T22:53:04Z","updated_at":"2015-04-12T22:32:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/derekparker/delve/pulls/88","html_url":"https://github.com/derekparker/delve/pull/88","diff_url":"https://github.com/derekparker/delve/pull/88.diff","patch_url":"https://github.com/derekparker/delve/pull/88.patch"},"body":"**NOTE** This is a work-in-progress, and the PR is to facilitate ongoing design discussion. This PR is intended to address #22.\r\n\r\n### Goals\r\n\r\n1. Introduce an HTTP server using `net.Listener` to provide a JSON-based remote API to the debugger\r\n  1. Support RESTful operations for state retrieval and commands\r\n  2. Support a long-polling watch requests to support event notifications\r\n2. Provide a Go client interface and default HTTP implementation\r\n4. Refactor the terminal to be a pure API client consumer\r\n\r\nThe original PR is based on [this comment](https://github.com/derekparker/delve/issues/22#issuecomment-84154167) on #22.\r\n"},"comment":{"url":"https://api.github.com/repos/derekparker/delve/issues/comments/92144981","html_url":"https://github.com/derekparker/delve/pull/88#issuecomment-92144981","issue_url":"https://api.github.com/repos/derekparker/delve/issues/88","id":92144981,"user":{"login":"derekparker","id":1314079,"avatar_url":"https://avatars.githubusercontent.com/u/1314079?v=3","gravatar_id":"","url":"https://api.github.com/users/derekparker","html_url":"https://github.com/derekparker","followers_url":"https://api.github.com/users/derekparker/followers","following_url":"https://api.github.com/users/derekparker/following{/other_user}","gists_url":"https://api.github.com/users/derekparker/gists{/gist_id}","starred_url":"https://api.github.com/users/derekparker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/derekparker/subscriptions","organizations_url":"https://api.github.com/users/derekparker/orgs","repos_url":"https://api.github.com/users/derekparker/repos","events_url":"https://api.github.com/users/derekparker/events{/privacy}","received_events_url":"https://api.github.com/users/derekparker/received_events","type":"User","site_admin":false},"created_at":"2015-04-12T22:32:30Z","updated_at":"2015-04-12T22:32:30Z","body":"@ironcladlou yeah, I'm taking a look into both now, have some changes in mind.\r\n\r\nIt looks like you've gotten all of the current functionality covered in this refactor.  I feel like this PR could be merged in this week, barring any huge issues.\r\n\r\nI've been thinking about testing, and haven't landed on solid ground there. My current thought process is that all tests in `proctl_test.go` should be converted into using a client, and not DebuggedProcess directly. To me it makes sense because that file is _integration style_ tests anyways, and that would nicely cover testing this refactor.\r\n\r\nThoughts?"}}