dathere / datapusher-plus

A standalone web service that pushes data into the CKAN Datastore fast & reliably. It pushes real good!

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

CKAN__DATAPUSHER__CALLBACK_URL_BASE ignored for data fetch

jhbruhn opened this issue · comments

I am using ckan behind a proxy which uses another internal authentication. Datapusher Plus (on the fixing-issues) branch is mostly aware of the CKAN__DATAPUSHER__CALLBACK_URL_BASE, but seems to be missing the DATAPUSHER_REWRITE_RESOURCES parameter present in the original datapusher implementation. This leads to an error while fetching data from the CKAN instance, while ironically the worker is on the same host/container in my case.

'original_url': resource_dict.get('url'),

The actual location where the URL should be rewritten is here:

resource_url = resource.get("url")

@jhbruhn thanks agian for your findings

Feel free to provide PR's for any issue that you could find