{"action":"created","issue":{"url":"https://api.github.com/repos/omegat-org/omegat/issues/12","repository_url":"https://api.github.com/repos/omegat-org/omegat","labels_url":"https://api.github.com/repos/omegat-org/omegat/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/omegat-org/omegat/issues/12/comments","events_url":"https://api.github.com/repos/omegat-org/omegat/issues/12/events","html_url":"https://github.com/omegat-org/omegat/pull/12","id":178646453,"number":12,"title":"LanguageTool network bridge squash","user":{"login":"amake","id":2172537,"avatar_url":"https://avatars.githubusercontent.com/u/2172537?v=3","gravatar_id":"","url":"https://api.github.com/users/amake","html_url":"https://github.com/amake","followers_url":"https://api.github.com/users/amake/followers","following_url":"https://api.github.com/users/amake/following{/other_user}","gists_url":"https://api.github.com/users/amake/gists{/gist_id}","starred_url":"https://api.github.com/users/amake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amake/subscriptions","organizations_url":"https://api.github.com/users/amake/orgs","repos_url":"https://api.github.com/users/amake/repos","events_url":"https://api.github.com/users/amake/events{/privacy}","received_events_url":"https://api.github.com/users/amake/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-09-22T15:45:01Z","updated_at":"2016-09-29T14:21:31Z","closed_at":"2016-09-29T14:21:30Z","pull_request":{"url":"https://api.github.com/repos/omegat-org/omegat/pulls/12","html_url":"https://github.com/omegat-org/omegat/pull/12","diff_url":"https://github.com/omegat-org/omegat/pull/12.diff","patch_url":"https://github.com/omegat-org/omegat/pull/12.patch"},"body":"This is a squash of #11 in preparation for commit to svn trunk.\r\n\r\nChanges vs #11:\r\n\r\n- The check on the remote server only looked at the response headers, but I found that it's easy to supply a URL for the old XML API, in which case communication will fail when we try to parse the JSON response. The check now looks to make sure the server is not sending XML.\r\n- Enabling/disabling rules on the native bridge was only affecting the LT instance in the thread that made the change, so issue checking (which happens on a special thread pool) did not see the rule changes at all. I fixed this but my solution is very unsatisfying.\r\n\r\nI haven't really tested the network bridge beyond the unit tests, so I will do that before committing. That should give me a better idea of how important spam prevention will be."},"comment":{"url":"https://api.github.com/repos/omegat-org/omegat/issues/comments/250480372","html_url":"https://github.com/omegat-org/omegat/pull/12#issuecomment-250480372","issue_url":"https://api.github.com/repos/omegat-org/omegat/issues/12","id":250480372,"user":{"login":"amake","id":2172537,"avatar_url":"https://avatars.githubusercontent.com/u/2172537?v=3","gravatar_id":"","url":"https://api.github.com/users/amake","html_url":"https://github.com/amake","followers_url":"https://api.github.com/users/amake/followers","following_url":"https://api.github.com/users/amake/following{/other_user}","gists_url":"https://api.github.com/users/amake/gists{/gist_id}","starred_url":"https://api.github.com/users/amake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amake/subscriptions","organizations_url":"https://api.github.com/users/amake/orgs","repos_url":"https://api.github.com/users/amake/repos","events_url":"https://api.github.com/users/amake/events{/privacy}","received_events_url":"https://api.github.com/users/amake/received_events","type":"User","site_admin":false},"created_at":"2016-09-29T14:21:30Z","updated_at":"2016-09-29T14:21:30Z","body":"This is now committed to trunk, so I will close this PR.\r\n\r\nThanks for all your hard work.\r\n\r\nIf you have any more changes to make please open a PR any time."}}
{"actor":{"display_login":"amake"}}