{"action":"opened","issue":{"url":"https://api.github.com/repos/okfn/ckanext-s3filestore/issues/2","labels_url":"https://api.github.com/repos/okfn/ckanext-s3filestore/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/okfn/ckanext-s3filestore/issues/2/comments","events_url":"https://api.github.com/repos/okfn/ckanext-s3filestore/issues/2/events","html_url":"https://github.com/okfn/ckanext-s3filestore/issues/2","id":106794007,"number":2,"title":"Extension still requires ckan.storage_path to be set up and be writable","user":{"login":"amercader","id":200230,"avatar_url":"https://avatars.githubusercontent.com/u/200230?v=3","gravatar_id":"","url":"https://api.github.com/users/amercader","html_url":"https://github.com/amercader","followers_url":"https://api.github.com/users/amercader/followers","following_url":"https://api.github.com/users/amercader/following{/other_user}","gists_url":"https://api.github.com/users/amercader/gists{/gist_id}","starred_url":"https://api.github.com/users/amercader/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amercader/subscriptions","organizations_url":"https://api.github.com/users/amercader/orgs","repos_url":"https://api.github.com/users/amercader/repos","events_url":"https://api.github.com/users/amercader/events{/privacy}","received_events_url":"https://api.github.com/users/amercader/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-09-16T15:09:28Z","updated_at":"2015-09-16T15:09:28Z","closed_at":null,"body":"This probably needs fixing in CKAN core, but flagging here for now. Because of various direct calls to the `uploader.get_storage_path()` method around CKAN, you still need to provide a `ckan.storage_path` config option even if you want to use s3 exclusively.\r\n\r\nEg:\r\n\r\n* On middleware.py: [here](https://github.com/ckan/ckan/blob/master/ckan/config/middleware.py#L149) a `storage` folder is created on the main uploads dir\r\n* On helpers.py [`uploads_enabled`](https://github.com/ckan/ckan/blob/master/ckan/lib/helpers.py#L1957). This is used to show the Upload button on the UI.\r\n\r\nThere might be others. It would be good to refactor those so they use `get_uploader` or whatever instead so they consider extensions"}}