-
Notifications
You must be signed in to change notification settings - Fork 54
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
Allow to provide an alternative CKAN host #226
Comments
I'm a little confused about why the |
There can be network restrictions or reverse proxies that in prevent accessing "outside" resources from the web server. A config option like this lets admins provide an alternate internal name/address that can be used to connect to ckan. |
Wouldn't that make the XLoader essentially unusable? I suppose it could still parse uploaded files, just not links. |
The xloader currently uses CKAN URLs for two things:
https://myckan.site.org/dataset/{id}/resource/{id}/download
)xloader_hook
action via a POST request. This was a surprise, as I was under the impression that xloader did not communicate via HTTP with the CKAN API.In any case, both these requests use the URL built with
ckan.site_url
, which is the public facing URL. This won't work on scenarios where the xloader has no access to it, for instance when running in a separate container in a docker compose setup. We need to introduce support for a new config option similar to ckan.datapusher.callback_url_base which will allow xloader to call e.g.http://ckan:5000/dataset/{id}/resource/{id}/download
instead ofhttps://myckan.site.org/dataset/{id}/resource/{id}/download
ckanext.xloader.callback_url_base
option to the config declarationresult_url
here, so that's the place to replace the host as well if the config option is presentThe text was updated successfully, but these errors were encountered: