{"action":"created","issue":{"url":"https://api.github.com/repos/puppetlabs/puppet/issues/3883","labels_url":"https://api.github.com/repos/puppetlabs/puppet/issues/3883/labels{/name}","comments_url":"https://api.github.com/repos/puppetlabs/puppet/issues/3883/comments","events_url":"https://api.github.com/repos/puppetlabs/puppet/issues/3883/events","html_url":"https://github.com/puppetlabs/puppet/pull/3883","id":72306875,"number":3883,"title":"(PUP-4420) Ensure pluginfacts don't ignore source permissions","user":{"login":"whopper","id":507621,"avatar_url":"https://avatars.githubusercontent.com/u/507621?v=3","gravatar_id":"","url":"https://api.github.com/users/whopper","html_url":"https://github.com/whopper","followers_url":"https://api.github.com/users/whopper/followers","following_url":"https://api.github.com/users/whopper/following{/other_user}","gists_url":"https://api.github.com/users/whopper/gists{/gist_id}","starred_url":"https://api.github.com/users/whopper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whopper/subscriptions","organizations_url":"https://api.github.com/users/whopper/orgs","repos_url":"https://api.github.com/users/whopper/repos","events_url":"https://api.github.com/users/whopper/events{/privacy}","received_events_url":"https://api.github.com/users/whopper/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/puppetlabs/puppet/labels/Client","name":"Client","color":"e11d21"},{"url":"https://api.github.com/repos/puppetlabs/puppet/labels/PL","name":"PL","color":"d4c5f9"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-04-30T23:16:24Z","updated_at":"2015-05-12T00:26:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/puppetlabs/puppet/pulls/3883","html_url":"https://github.com/puppetlabs/puppet/pull/3883","diff_url":"https://github.com/puppetlabs/puppet/pull/3883.diff","patch_url":"https://github.com/puppetlabs/puppet/pull/3883.patch"},"body":"Prior to this commit, the :source_permissions attribute was\r\nnot being properly propagated into the Posix MetaStat class,\r\nand so source permissions were ignored for all resources.\r\n\r\nExternal facts are required to use source permissions so\r\nthat the facts are executable when they are downloaded onto\r\nthe agent. This means that due to this regression, external\r\nfacts were being pluginsync'd to the client without execution\r\npermissions.\r\n\r\nThis commit re-enables using source permissions for fact\r\npluginsync, while continuing to ignore them otherwise.\r\n\r\nSee commit 21edcd6 for the first fix for of a similar, earlier\r\nregression."},"comment":{"url":"https://api.github.com/repos/puppetlabs/puppet/issues/comments/101080671","html_url":"https://github.com/puppetlabs/puppet/pull/3883#issuecomment-101080671","issue_url":"https://api.github.com/repos/puppetlabs/puppet/issues/3883","id":101080671,"user":{"login":"whopper","id":507621,"avatar_url":"https://avatars.githubusercontent.com/u/507621?v=3","gravatar_id":"","url":"https://api.github.com/users/whopper","html_url":"https://github.com/whopper","followers_url":"https://api.github.com/users/whopper/followers","following_url":"https://api.github.com/users/whopper/following{/other_user}","gists_url":"https://api.github.com/users/whopper/gists{/gist_id}","starred_url":"https://api.github.com/users/whopper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whopper/subscriptions","organizations_url":"https://api.github.com/users/whopper/orgs","repos_url":"https://api.github.com/users/whopper/repos","events_url":"https://api.github.com/users/whopper/events{/privacy}","received_events_url":"https://api.github.com/users/whopper/received_events","type":"User","site_admin":false},"created_at":"2015-05-12T00:26:54Z","updated_at":"2015-05-12T00:26:54Z","body":"@joshcooper pushed your suggested change but I won't have a chance to test it / see if there are other needed changes until tomorrow D:"}}