{"action":"created","issue":{"url":"https://api.github.com/repos/ckan/ckan/issues/2570","labels_url":"https://api.github.com/repos/ckan/ckan/issues/2570/labels{/name}","comments_url":"https://api.github.com/repos/ckan/ckan/issues/2570/comments","events_url":"https://api.github.com/repos/ckan/ckan/issues/2570/events","html_url":"https://github.com/ckan/ckan/pull/2570","id":99396723,"number":2570,"title":"[WIP] 2564-2562-Auth fixes for datastore_search and datastore_search_sql ","user":{"login":"rossjones","id":118063,"avatar_url":"https://avatars.githubusercontent.com/u/118063?v=3","gravatar_id":"","url":"https://api.github.com/users/rossjones","html_url":"https://github.com/rossjones","followers_url":"https://api.github.com/users/rossjones/followers","following_url":"https://api.github.com/users/rossjones/following{/other_user}","gists_url":"https://api.github.com/users/rossjones/gists{/gist_id}","starred_url":"https://api.github.com/users/rossjones/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rossjones/subscriptions","organizations_url":"https://api.github.com/users/rossjones/orgs","repos_url":"https://api.github.com/users/rossjones/repos","events_url":"https://api.github.com/users/rossjones/events{/privacy}","received_events_url":"https://api.github.com/users/rossjones/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-08-06T09:31:47Z","updated_at":"2015-09-07T18:08:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ckan/ckan/pulls/2570","html_url":"https://github.com/ckan/ckan/pull/2570","diff_url":"https://github.com/ckan/ckan/pull/2570.diff","patch_url":"https://github.com/ckan/ckan/pull/2570.patch"},"body":"This PR removes the use of the write_url in the datastore_search action, as the permissions are already checked later on against the auth.datastore_search function (which delegates to resource_show).\r\n\r\nLooks like the non-removal of write_url in the action method was an accidental oversight.\r\n\r\nTests (including the privacy tests for TestDatastoreSearch.test_search_private_dataset) continue to pass.\r\n\r\nShould fix tickets #2564 and #2562 "},"comment":{"url":"https://api.github.com/repos/ckan/ckan/issues/comments/138350649","html_url":"https://github.com/ckan/ckan/pull/2570#issuecomment-138350649","issue_url":"https://api.github.com/repos/ckan/ckan/issues/2570","id":138350649,"user":{"login":"rossjones","id":118063,"avatar_url":"https://avatars.githubusercontent.com/u/118063?v=3","gravatar_id":"","url":"https://api.github.com/users/rossjones","html_url":"https://github.com/rossjones","followers_url":"https://api.github.com/users/rossjones/followers","following_url":"https://api.github.com/users/rossjones/following{/other_user}","gists_url":"https://api.github.com/users/rossjones/gists{/gist_id}","starred_url":"https://api.github.com/users/rossjones/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rossjones/subscriptions","organizations_url":"https://api.github.com/users/rossjones/orgs","repos_url":"https://api.github.com/users/rossjones/repos","events_url":"https://api.github.com/users/rossjones/events{/privacy}","received_events_url":"https://api.github.com/users/rossjones/received_events","type":"User","site_admin":false},"created_at":"2015-09-07T18:08:09Z","updated_at":"2015-09-07T18:08:09Z","body":"Missing config options?"}}