Enable path-based interactive tools

usegalaxy-eu / infrastructure-playbook

Ansible playbook for managing UseGalaxy.eu infrastructure.

MIT License

16 stars 91 forks source link

Enable path-based interactive tools #1256

Closed kysrpex closed 1 month ago

kysrpex commented 1 month ago

Enable access via path (e.g. https://usegalaxy.eu/interactivetool/ep/3qg8nstfnl44r/fsjbqy66kln5) to interactive tools that support it (those for which requires_domain="False" in the tool's xml file).

More information:

nginx configuration: https://docs.galaxyproject.org/en/master/admin/special_topics/interactivetools.html#nginx-proxy-server-configuration-in-production
interactive tools proxy configuration: https://github.com/galaxyproject/galaxy/pull/18481#issuecomment-2217313953, https://github.com/galaxyproject/gx-it-proxy/blob/v0.0.6/lib/proxy.js#L101-L141, https://github.com/galaxyproject/gx-it-proxy/blob/v0.0.6/lib/main.js#L16

Related commit: usegalaxy-eu/galaxy@e216b45

kysrpex commented 1 month ago

Ping @sveinugu (just FYI)

sveinugu commented 1 month ago

Great!

Could you also update the ansible playbook example you mentioned?

bgruening commented 1 month ago

Interesting! Thanks @kysrpex! If this gets merged I will remove my commit in our Galaxy fork and we can try.

kysrpex commented 1 month ago

Great!

Could you also update the ansible playbook example you mentioned?

@sveinugu Commit https://github.com/usegalaxy-eu/infrastructure-playbook/pull/1256/commits/a97941bff0e9d032eae7f3e807d5c36bcadeab31 would not be valid without having updated the Ansible role, see the latest release from 2h ago and the comment on the new README.

# configuring this path prefix enables path-based interactive tools
# (https://docs.galaxyproject.org/en/master/admin/special_topics/interactivetools.html#nginx-proxy-server-configuration-in-production)
gie_proxy_path_prefix: /interactivetool/ep

bgruening commented 1 month ago

So if someone presses merge here, I will remove the commit in the galaxy repo and redeploy :)