{"action":"created","issue":{"url":"https://api.github.com/repos/uProxy/uproxy-lib/issues/332","labels_url":"https://api.github.com/repos/uProxy/uproxy-lib/issues/332/labels{/name}","comments_url":"https://api.github.com/repos/uProxy/uproxy-lib/issues/332/comments","events_url":"https://api.github.com/repos/uProxy/uproxy-lib/issues/332/events","html_url":"https://github.com/uProxy/uproxy-lib/pull/332","id":121059722,"number":332,"title":"better error handling and logging for cloud","user":{"login":"trevj","id":5142116,"avatar_url":"https://avatars.githubusercontent.com/u/5142116?v=3","gravatar_id":"","url":"https://api.github.com/users/trevj","html_url":"https://github.com/trevj","followers_url":"https://api.github.com/users/trevj/followers","following_url":"https://api.github.com/users/trevj/following{/other_user}","gists_url":"https://api.github.com/users/trevj/gists{/gist_id}","starred_url":"https://api.github.com/users/trevj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/trevj/subscriptions","organizations_url":"https://api.github.com/users/trevj/orgs","repos_url":"https://api.github.com/users/trevj/repos","events_url":"https://api.github.com/users/trevj/events{/privacy}","received_events_url":"https://api.github.com/users/trevj/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"lucyhe","id":8723127,"avatar_url":"https://avatars.githubusercontent.com/u/8723127?v=3","gravatar_id":"","url":"https://api.github.com/users/lucyhe","html_url":"https://github.com/lucyhe","followers_url":"https://api.github.com/users/lucyhe/followers","following_url":"https://api.github.com/users/lucyhe/following{/other_user}","gists_url":"https://api.github.com/users/lucyhe/gists{/gist_id}","starred_url":"https://api.github.com/users/lucyhe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lucyhe/subscriptions","organizations_url":"https://api.github.com/users/lucyhe/orgs","repos_url":"https://api.github.com/users/lucyhe/repos","events_url":"https://api.github.com/users/lucyhe/events{/privacy}","received_events_url":"https://api.github.com/users/lucyhe/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-12-08T17:39:43Z","updated_at":"2015-12-09T21:19:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/uProxy/uproxy-lib/pulls/332","html_url":"https://github.com/uProxy/uproxy-lib/pull/332","diff_url":"https://github.com/uProxy/uproxy-lib/pull/332.diff","patch_url":"https://github.com/uProxy/uproxy-lib/pull/332.patch"},"body":"A bunch of stuff which, from tesing on my workstation and laptop, seems to significantly help the connection rate:\r\n * factor out a class, `LineFeeder`, to convert an `ArrayBuffer` queue -> strings, delimiting on newlines (this was a real mess...#1 fix here)\r\n * moar zork logging!\r\n * always reject promises with an `Error`\r\n * return after rejecting, when necessary\n\n<!-- Reviewable:start -->\n[<img src=\"https://reviewable.io/review_button.png\" height=40 alt=\"Review on Reviewable\"/>](https://reviewable.io/reviews/uproxy/uproxy-lib/332)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/uProxy/uproxy-lib/issues/comments/163394863","html_url":"https://github.com/uProxy/uproxy-lib/pull/332#issuecomment-163394863","issue_url":"https://api.github.com/repos/uProxy/uproxy-lib/issues/332","id":163394863,"user":{"login":"trevj","id":5142116,"avatar_url":"https://avatars.githubusercontent.com/u/5142116?v=3","gravatar_id":"","url":"https://api.github.com/users/trevj","html_url":"https://github.com/trevj","followers_url":"https://api.github.com/users/trevj/followers","following_url":"https://api.github.com/users/trevj/following{/other_user}","gists_url":"https://api.github.com/users/trevj/gists{/gist_id}","starred_url":"https://api.github.com/users/trevj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/trevj/subscriptions","organizations_url":"https://api.github.com/users/trevj/orgs","repos_url":"https://api.github.com/users/trevj/repos","events_url":"https://api.github.com/users/trevj/events{/privacy}","received_events_url":"https://api.github.com/users/trevj/received_events","type":"User","site_admin":false},"created_at":"2015-12-09T21:19:49Z","updated_at":"2015-12-09T21:19:49Z","body":"\n\n\n\nReview status: 4 of 5 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.\n\n---\n\n*[src/zork/freedom-module.ts, line 79 \\[r1\\]](https://reviewable.io:443/reviews/uproxy/uproxy-lib/332#-K52efix20xSmwC5qA1u:-K57JtNIjrGOb9cye2RP:-1013429418) ([raw file](https://github.com/uproxy/uproxy-lib/blob/4f367c39f500c745c4f572dfd6c40b8129feeda4/src/zork/freedom-module.ts#L79)):*\nQuite right! It wasn't even camelcase. Renamed to `processCommand`.\n\n---\n\n\n*Comments from the [review on Reviewable.io](https://reviewable.io:443/reviews/uproxy/uproxy-lib/332)*\n<!-- Sent from Reviewable.io -->\n"}}