{"action":"created","issue":{"url":"https://api.github.com/repos/jenkinsci/remoting/issues/56","labels_url":"https://api.github.com/repos/jenkinsci/remoting/issues/56/labels{/name}","comments_url":"https://api.github.com/repos/jenkinsci/remoting/issues/56/comments","events_url":"https://api.github.com/repos/jenkinsci/remoting/issues/56/events","html_url":"https://github.com/jenkinsci/remoting/pull/56","id":98794332,"number":56,"title":"Trying to guide people away from implementing RoleSensitive manually","user":{"login":"jglick","id":154109,"avatar_url":"https://avatars.githubusercontent.com/u/154109?v=3","gravatar_id":"","url":"https://api.github.com/users/jglick","html_url":"https://github.com/jglick","followers_url":"https://api.github.com/users/jglick/followers","following_url":"https://api.github.com/users/jglick/following{/other_user}","gists_url":"https://api.github.com/users/jglick/gists{/gist_id}","starred_url":"https://api.github.com/users/jglick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jglick/subscriptions","organizations_url":"https://api.github.com/users/jglick/orgs","repos_url":"https://api.github.com/users/jglick/repos","events_url":"https://api.github.com/users/jglick/events{/privacy}","received_events_url":"https://api.github.com/users/jglick/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-08-03T17:01:19Z","updated_at":"2015-08-03T17:02:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jenkinsci/remoting/pulls/56","html_url":"https://github.com/jenkinsci/remoting/pull/56","diff_url":"https://github.com/jenkinsci/remoting/pull/56.diff","patch_url":"https://github.com/jenkinsci/remoting/pull/56.patch"},"body":"After about the seventeenth Jenkins plugin pull request (most recently https://github.com/jenkinsci/analysis-core-plugin/pull/31) featuring something like\r\n\r\n```java\r\n@Override\r\npublic void checkRoles(RoleChecker checker) throws SecurityException {\r\n    // TODO what do we need to do here?\r\n}\r\n```\r\n\r\nI am trying to guide people to the answer, in case they read Javadoc anyway.\r\n\r\n(For SECURITY-144 we explicitly decided to introduce `RoleSensitive` despite the source incompatibility, while keeping a binary-compatible runtime behavior, since every plugin author really must decide how a given callable is _intended_ to be used.)\r\n\r\n@reviewbybees"},"comment":{"url":"https://api.github.com/repos/jenkinsci/remoting/issues/comments/127332004","html_url":"https://github.com/jenkinsci/remoting/pull/56#issuecomment-127332004","issue_url":"https://api.github.com/repos/jenkinsci/remoting/issues/56","id":127332004,"user":{"login":"reviewbybees","id":10512980,"avatar_url":"https://avatars.githubusercontent.com/u/10512980?v=3","gravatar_id":"","url":"https://api.github.com/users/reviewbybees","html_url":"https://github.com/reviewbybees","followers_url":"https://api.github.com/users/reviewbybees/followers","following_url":"https://api.github.com/users/reviewbybees/following{/other_user}","gists_url":"https://api.github.com/users/reviewbybees/gists{/gist_id}","starred_url":"https://api.github.com/users/reviewbybees/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/reviewbybees/subscriptions","organizations_url":"https://api.github.com/users/reviewbybees/orgs","repos_url":"https://api.github.com/users/reviewbybees/repos","events_url":"https://api.github.com/users/reviewbybees/events{/privacy}","received_events_url":"https://api.github.com/users/reviewbybees/received_events","type":"User","site_admin":false},"created_at":"2015-08-03T17:02:01Z","updated_at":"2015-08-03T17:02:01Z","body":"This pull request originates from a [CloudBees](https://www.cloudbees.com/) employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see [this explanation](https://github.com/reviewbybees/about#about-reviewbybees).\n"}}