{"action":"created","issue":{"url":"https://api.github.com/repos/chef/chef-server/issues/248","labels_url":"https://api.github.com/repos/chef/chef-server/issues/248/labels{/name}","comments_url":"https://api.github.com/repos/chef/chef-server/issues/248/comments","events_url":"https://api.github.com/repos/chef/chef-server/issues/248/events","html_url":"https://github.com/chef/chef-server/pull/248","id":75355877,"number":248,"title":"ACL Restrictions - Prevent admins group removal from grant ACE","user":{"login":"mmzyk","id":66784,"avatar_url":"https://avatars.githubusercontent.com/u/66784?v=3","gravatar_id":"","url":"https://api.github.com/users/mmzyk","html_url":"https://github.com/mmzyk","followers_url":"https://api.github.com/users/mmzyk/followers","following_url":"https://api.github.com/users/mmzyk/following{/other_user}","gists_url":"https://api.github.com/users/mmzyk/gists{/gist_id}","starred_url":"https://api.github.com/users/mmzyk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmzyk/subscriptions","organizations_url":"https://api.github.com/users/mmzyk/orgs","repos_url":"https://api.github.com/users/mmzyk/repos","events_url":"https://api.github.com/users/mmzyk/events{/privacy}","received_events_url":"https://api.github.com/users/mmzyk/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/chef/chef-server/labels/in%20progress","name":"in progress","color":"ededed"},{"url":"https://api.github.com/repos/chef/chef-server/labels/Signed%20CLA","name":"Signed CLA","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":17,"created_at":"2015-05-11T21:00:13Z","updated_at":"2015-07-13T16:18:26Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/chef/chef-server/pulls/248","html_url":"https://github.com/chef/chef-server/pull/248","diff_url":"https://github.com/chef/chef-server/pull/248.diff","patch_url":"https://github.com/chef/chef-server/pull/248.patch"},"body":"Add in stubb for performing acl integrity checks\r\n\r\nThis commit changes nothing about how the current code operates. However, it does add a new module and call to that module that gives a hook for performing acl integrity checks, such as if we are removing the admin group from the grant ace, which is something we desire not to be possible.\r\n\r\nThis branch is branched from https://github.com/chef/chef-server/pull/240.\r\n\r\nThe desire at this point is that both of these branches could be merged to master and would essential be no-ops as the desired functionality is built on top of these changes."},"comment":{"url":"https://api.github.com/repos/chef/chef-server/issues/comments/120984544","html_url":"https://github.com/chef/chef-server/pull/248#issuecomment-120984544","issue_url":"https://api.github.com/repos/chef/chef-server/issues/248","id":120984544,"user":{"login":"mmzyk","id":66784,"avatar_url":"https://avatars.githubusercontent.com/u/66784?v=3","gravatar_id":"","url":"https://api.github.com/users/mmzyk","html_url":"https://github.com/mmzyk","followers_url":"https://api.github.com/users/mmzyk/followers","following_url":"https://api.github.com/users/mmzyk/following{/other_user}","gists_url":"https://api.github.com/users/mmzyk/gists{/gist_id}","starred_url":"https://api.github.com/users/mmzyk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmzyk/subscriptions","organizations_url":"https://api.github.com/users/mmzyk/orgs","repos_url":"https://api.github.com/users/mmzyk/repos","events_url":"https://api.github.com/users/mmzyk/events{/privacy}","received_events_url":"https://api.github.com/users/mmzyk/received_events","type":"User","site_admin":false},"created_at":"2015-07-13T16:18:26Z","updated_at":"2015-07-13T16:18:26Z","body":"Remain questions, other than general review on this PR:\r\n\r\nDoes this work need to incorporate/change the API version in anyway? I'm not sure how we're handling that. I assume no, but want to check.\r\n\r\nShould this go out in the next server release after it goes down, or do we need to wait for another release (Chef server 13? 12.2? something else?)?"}}