{"action":"created","issue":{"url":"https://api.github.com/repos/ckan/ckan/issues/2846","repository_url":"https://api.github.com/repos/ckan/ckan","labels_url":"https://api.github.com/repos/ckan/ckan/issues/2846/labels{/name}","comments_url":"https://api.github.com/repos/ckan/ckan/issues/2846/comments","events_url":"https://api.github.com/repos/ckan/ckan/issues/2846/events","html_url":"https://github.com/ckan/ckan/pull/2846","id":128858522,"number":2846,"title":"Use 403 when actions are forbidden, not 401 ","user":{"login":"wardi","id":153258,"avatar_url":"https://avatars.githubusercontent.com/u/153258?v=3","gravatar_id":"","url":"https://api.github.com/users/wardi","html_url":"https://github.com/wardi","followers_url":"https://api.github.com/users/wardi/followers","following_url":"https://api.github.com/users/wardi/following{/other_user}","gists_url":"https://api.github.com/users/wardi/gists{/gist_id}","starred_url":"https://api.github.com/users/wardi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wardi/subscriptions","organizations_url":"https://api.github.com/users/wardi/orgs","repos_url":"https://api.github.com/users/wardi/repos","events_url":"https://api.github.com/users/wardi/events{/privacy}","received_events_url":"https://api.github.com/users/wardi/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"nigelbabu","id":373438,"avatar_url":"https://avatars.githubusercontent.com/u/373438?v=3","gravatar_id":"","url":"https://api.github.com/users/nigelbabu","html_url":"https://github.com/nigelbabu","followers_url":"https://api.github.com/users/nigelbabu/followers","following_url":"https://api.github.com/users/nigelbabu/following{/other_user}","gists_url":"https://api.github.com/users/nigelbabu/gists{/gist_id}","starred_url":"https://api.github.com/users/nigelbabu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nigelbabu/subscriptions","organizations_url":"https://api.github.com/users/nigelbabu/orgs","repos_url":"https://api.github.com/users/nigelbabu/repos","events_url":"https://api.github.com/users/nigelbabu/events{/privacy}","received_events_url":"https://api.github.com/users/nigelbabu/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2016-01-26T15:42:11Z","updated_at":"2016-03-28T14:26:51Z","closed_at":"2016-03-28T06:22:54Z","pull_request":{"url":"https://api.github.com/repos/ckan/ckan/pulls/2846","html_url":"https://github.com/ckan/ckan/pull/2846","diff_url":"https://github.com/ckan/ckan/pull/2846.diff","patch_url":"https://github.com/ckan/ckan/pull/2846.patch"},"body":"401 is for when a user can't authenticate. All our current 401 errors are causing users to be logged out by repoze (which is reasonable). Instead we want to say \"yes, I know who you are, you just aren't allowed to access this page\""},"comment":{"url":"https://api.github.com/repos/ckan/ckan/issues/comments/202415919","html_url":"https://github.com/ckan/ckan/pull/2846#issuecomment-202415919","issue_url":"https://api.github.com/repos/ckan/ckan/issues/2846","id":202415919,"user":{"login":"wardi","id":153258,"avatar_url":"https://avatars.githubusercontent.com/u/153258?v=3","gravatar_id":"","url":"https://api.github.com/users/wardi","html_url":"https://github.com/wardi","followers_url":"https://api.github.com/users/wardi/followers","following_url":"https://api.github.com/users/wardi/following{/other_user}","gists_url":"https://api.github.com/users/wardi/gists{/gist_id}","starred_url":"https://api.github.com/users/wardi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wardi/subscriptions","organizations_url":"https://api.github.com/users/wardi/orgs","repos_url":"https://api.github.com/users/wardi/repos","events_url":"https://api.github.com/users/wardi/events{/privacy}","received_events_url":"https://api.github.com/users/wardi/received_events","type":"User","site_admin":false},"created_at":"2016-03-28T14:26:51Z","updated_at":"2016-03-28T14:26:51Z","body":"@nigelbabu thanks! I'll look into the test failures, possibly some new tests added since I created this PR"}}