{"action":"created","issue":{"url":"https://api.github.com/repos/naparuba/shinken/issues/1473","labels_url":"https://api.github.com/repos/naparuba/shinken/issues/1473/labels{/name}","comments_url":"https://api.github.com/repos/naparuba/shinken/issues/1473/comments","events_url":"https://api.github.com/repos/naparuba/shinken/issues/1473/events","html_url":"https://github.com/naparuba/shinken/pull/1473","id":55521786,"number":1473,"title":"Disable SSLv3 for CherryPy backend","user":{"login":"colourmeamused","id":4123445,"avatar_url":"https://avatars.githubusercontent.com/u/4123445?v=3","gravatar_id":"","url":"https://api.github.com/users/colourmeamused","html_url":"https://github.com/colourmeamused","followers_url":"https://api.github.com/users/colourmeamused/followers","following_url":"https://api.github.com/users/colourmeamused/following{/other_user}","gists_url":"https://api.github.com/users/colourmeamused/gists{/gist_id}","starred_url":"https://api.github.com/users/colourmeamused/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/colourmeamused/subscriptions","organizations_url":"https://api.github.com/users/colourmeamused/orgs","repos_url":"https://api.github.com/users/colourmeamused/repos","events_url":"https://api.github.com/users/colourmeamused/events{/privacy}","received_events_url":"https://api.github.com/users/colourmeamused/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-01-26T18:55:18Z","updated_at":"2015-01-29T18:24:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/naparuba/shinken/pulls/1473","html_url":"https://github.com/naparuba/shinken/pull/1473","diff_url":"https://github.com/naparuba/shinken/pull/1473.diff","patch_url":"https://github.com/naparuba/shinken/pull/1473.patch"},"body":"Takes care of Poodle (https://www.poodlescan.com/) when SSL is used with cherrypy.\r\nMy instance was even allowing SSLv2, but that might be a Debian Wheezy issue.\r\nHappy to clean / improve code further based on suggestions."},"comment":{"url":"https://api.github.com/repos/naparuba/shinken/issues/comments/72076591","html_url":"https://github.com/naparuba/shinken/pull/1473#issuecomment-72076591","issue_url":"https://api.github.com/repos/naparuba/shinken/issues/1473","id":72076591,"user":{"login":"Seb-Solon","id":856137,"avatar_url":"https://avatars.githubusercontent.com/u/856137?v=3","gravatar_id":"","url":"https://api.github.com/users/Seb-Solon","html_url":"https://github.com/Seb-Solon","followers_url":"https://api.github.com/users/Seb-Solon/followers","following_url":"https://api.github.com/users/Seb-Solon/following{/other_user}","gists_url":"https://api.github.com/users/Seb-Solon/gists{/gist_id}","starred_url":"https://api.github.com/users/Seb-Solon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Seb-Solon/subscriptions","organizations_url":"https://api.github.com/users/Seb-Solon/orgs","repos_url":"https://api.github.com/users/Seb-Solon/repos","events_url":"https://api.github.com/users/Seb-Solon/events{/privacy}","received_events_url":"https://api.github.com/users/Seb-Solon/received_events","type":"User","site_admin":false},"created_at":"2015-01-29T18:24:17Z","updated_at":"2015-01-29T18:24:17Z","body":"I don't think we should merge this right now. The commit include some pep8 of current code. This will create conflict with the big pep8 pull request. \r\n\r\nThat's why having only one commit for a pull request is not a good idea when you are doing more that one thing. We should have like 1 commit for the fix, 1 for the pep8 and another one for the test (thanks for that by the way). It is really easy to read, and easy to understand. \r\n\r\n\r\n"}}