Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CKAN__DATAPUSHER__CALLBACK_URL_BASE ignored for data fetch #135

Open
jhbruhn opened this issue Mar 28, 2024 · 2 comments
Open

CKAN__DATAPUSHER__CALLBACK_URL_BASE ignored for data fetch #135

jhbruhn opened this issue Mar 28, 2024 · 2 comments

Comments

@jhbruhn
Copy link

jhbruhn commented Mar 28, 2024

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'),

@jhbruhn
Copy link
Author

jhbruhn commented Mar 28, 2024

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

resource_url = resource.get("url")

@tino097
Copy link
Collaborator

tino097 commented Mar 28, 2024

@jhbruhn thanks agian for your findings

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants