{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible/issues/12276","labels_url":"https://api.github.com/repos/ansible/ansible/issues/12276/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible/issues/12276/comments","events_url":"https://api.github.com/repos/ansible/ansible/issues/12276/events","html_url":"https://github.com/ansible/ansible/pull/12276","id":105310535,"number":12276,"title":"Implement locking/unlocking in ssh.py","user":{"login":"amenonsen","id":23628,"avatar_url":"https://avatars.githubusercontent.com/u/23628?v=3","gravatar_id":"","url":"https://api.github.com/users/amenonsen","html_url":"https://github.com/amenonsen","followers_url":"https://api.github.com/users/amenonsen/followers","following_url":"https://api.github.com/users/amenonsen/following{/other_user}","gists_url":"https://api.github.com/users/amenonsen/gists{/gist_id}","starred_url":"https://api.github.com/users/amenonsen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amenonsen/subscriptions","organizations_url":"https://api.github.com/users/amenonsen/orgs","repos_url":"https://api.github.com/users/amenonsen/repos","events_url":"https://api.github.com/users/amenonsen/events{/privacy}","received_events_url":"https://api.github.com/users/amenonsen/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ansible/ansible/labels/bugfix_pull_request","name":"bugfix_pull_request","color":"ded270"},{"url":"https://api.github.com/repos/ansible/ansible/labels/P2","name":"P2","color":"fad8c7"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/ansible/ansible/milestones/5","html_url":"https://github.com/ansible/ansible/milestones/v2","labels_url":"https://api.github.com/repos/ansible/ansible/milestones/5/labels","id":975800,"number":5,"title":"v2","description":null,"creator":{"login":"bcoca","id":836171,"avatar_url":"https://avatars.githubusercontent.com/u/836171?v=3","gravatar_id":"","url":"https://api.github.com/users/bcoca","html_url":"https://github.com/bcoca","followers_url":"https://api.github.com/users/bcoca/followers","following_url":"https://api.github.com/users/bcoca/following{/other_user}","gists_url":"https://api.github.com/users/bcoca/gists{/gist_id}","starred_url":"https://api.github.com/users/bcoca/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bcoca/subscriptions","organizations_url":"https://api.github.com/users/bcoca/orgs","repos_url":"https://api.github.com/users/bcoca/repos","events_url":"https://api.github.com/users/bcoca/events{/privacy}","received_events_url":"https://api.github.com/users/bcoca/received_events","type":"User","site_admin":false},"open_issues":23,"closed_issues":253,"state":"open","created_at":"2015-02-11T19:43:06Z","updated_at":"2015-09-08T22:34:43Z","due_on":null,"closed_at":null},"comments":2,"created_at":"2015-09-08T06:29:10Z","updated_at":"2015-09-09T00:01:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ansible/ansible/pulls/12276","html_url":"https://github.com/ansible/ansible/pull/12276","diff_url":"https://github.com/ansible/ansible/pull/12276.diff","patch_url":"https://github.com/ansible/ansible/pull/12276.patch"},"body":"I had to reorganise the internals of ``ssh.py`` to make the connection handling clearer. I've tested this with various combinations of ``ANSIBLE_PIPELINING=y/n``, ``ANSIBLE_SCP_IF_SSH=y/n``, ``become: y/n``, ``become_method: su/sudo``, etc. but more testing would be appreciated."},"comment":{"url":"https://api.github.com/repos/ansible/ansible/issues/comments/138735165","html_url":"https://github.com/ansible/ansible/pull/12276#issuecomment-138735165","issue_url":"https://api.github.com/repos/ansible/ansible/issues/12276","id":138735165,"user":{"login":"amenonsen","id":23628,"avatar_url":"https://avatars.githubusercontent.com/u/23628?v=3","gravatar_id":"","url":"https://api.github.com/users/amenonsen","html_url":"https://github.com/amenonsen","followers_url":"https://api.github.com/users/amenonsen/followers","following_url":"https://api.github.com/users/amenonsen/following{/other_user}","gists_url":"https://api.github.com/users/amenonsen/gists{/gist_id}","starred_url":"https://api.github.com/users/amenonsen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amenonsen/subscriptions","organizations_url":"https://api.github.com/users/amenonsen/orgs","repos_url":"https://api.github.com/users/amenonsen/repos","events_url":"https://api.github.com/users/amenonsen/events{/privacy}","received_events_url":"https://api.github.com/users/amenonsen/received_events","type":"User","site_admin":false},"created_at":"2015-09-09T00:01:25Z","updated_at":"2015-09-09T00:01:25Z","body":"@halberom many thanks for testing. This is caused by a confusion about what ``sudoable`` actually means; I had thought it (meant \"this task wants privilege escalation\", and therefore) wouldn't be set for ``put_file()`` and ``fetch_file()`` (i.e. scp/sftp), but subsequent discussions with @bcoca made it clear that this isn't the case. I'll fix it."}}