{"action":"created","issue":{"url":"https://api.github.com/repos/jenkinsci/remoting/issues/28","labels_url":"https://api.github.com/repos/jenkinsci/remoting/issues/28/labels{/name}","comments_url":"https://api.github.com/repos/jenkinsci/remoting/issues/28/comments","events_url":"https://api.github.com/repos/jenkinsci/remoting/issues/28/events","html_url":"https://github.com/jenkinsci/remoting/pull/28","id":55373596,"number":28,"title":"[JENKINS-26580] Refactor slave JNLP engine to make it easier to add more...","user":{"login":"akshayabd","id":120980,"avatar_url":"https://avatars.githubusercontent.com/u/120980?v=3","gravatar_id":"","url":"https://api.github.com/users/akshayabd","html_url":"https://github.com/akshayabd","followers_url":"https://api.github.com/users/akshayabd/followers","following_url":"https://api.github.com/users/akshayabd/following{/other_user}","gists_url":"https://api.github.com/users/akshayabd/gists{/gist_id}","starred_url":"https://api.github.com/users/akshayabd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akshayabd/subscriptions","organizations_url":"https://api.github.com/users/akshayabd/orgs","repos_url":"https://api.github.com/users/akshayabd/repos","events_url":"https://api.github.com/users/akshayabd/events{/privacy}","received_events_url":"https://api.github.com/users/akshayabd/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"kohsuke","id":50003,"avatar_url":"https://avatars.githubusercontent.com/u/50003?v=3","gravatar_id":"","url":"https://api.github.com/users/kohsuke","html_url":"https://github.com/kohsuke","followers_url":"https://api.github.com/users/kohsuke/followers","following_url":"https://api.github.com/users/kohsuke/following{/other_user}","gists_url":"https://api.github.com/users/kohsuke/gists{/gist_id}","starred_url":"https://api.github.com/users/kohsuke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kohsuke/subscriptions","organizations_url":"https://api.github.com/users/kohsuke/orgs","repos_url":"https://api.github.com/users/kohsuke/repos","events_url":"https://api.github.com/users/kohsuke/events{/privacy}","received_events_url":"https://api.github.com/users/kohsuke/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-01-24T15:02:13Z","updated_at":"2015-01-28T09:39:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jenkinsci/remoting/pulls/28","html_url":"https://github.com/jenkinsci/remoting/pull/28","diff_url":"https://github.com/jenkinsci/remoting/pull/28.diff","patch_url":"https://github.com/jenkinsci/remoting/pull/28.patch"},"body":"... protocols in the future. The engine will now call a factory that returns a list of protocols to try in order. Developers can implement new protocols in separate classes and add them to the factory to be used. Added tests for existing protocols. I noticed the Engine class doesn't have a corresponding Test class. It would be nice to add one, but I think it needs more refactoring before it would be realistic to add tests for it."},"comment":{"url":"https://api.github.com/repos/jenkinsci/remoting/issues/comments/71805933","html_url":"https://github.com/jenkinsci/remoting/pull/28#issuecomment-71805933","issue_url":"https://api.github.com/repos/jenkinsci/remoting/issues/28","id":71805933,"user":{"login":"akshayabd","id":120980,"avatar_url":"https://avatars.githubusercontent.com/u/120980?v=3","gravatar_id":"","url":"https://api.github.com/users/akshayabd","html_url":"https://github.com/akshayabd","followers_url":"https://api.github.com/users/akshayabd/followers","following_url":"https://api.github.com/users/akshayabd/following{/other_user}","gists_url":"https://api.github.com/users/akshayabd/gists{/gist_id}","starred_url":"https://api.github.com/users/akshayabd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akshayabd/subscriptions","organizations_url":"https://api.github.com/users/akshayabd/orgs","repos_url":"https://api.github.com/users/akshayabd/repos","events_url":"https://api.github.com/users/akshayabd/events{/privacy}","received_events_url":"https://api.github.com/users/akshayabd/received_events","type":"User","site_admin":false},"created_at":"2015-01-28T09:39:41Z","updated_at":"2015-01-28T09:39:41Z","body":"PTAL. Made updates based on feedback.\r\n\r\nI've tested the changes with 1.580.2 and 1.599-SNAPSHOT. I'm downloading older versions now to test - the downloads are taking several hours for some reason:\r\n\r\nhttp://mirrors.jenkins-ci.org/war-stable/1.466.1/\r\nhttp://mirrors.jenkins-ci.org/war/1.460/"}}