-
Notifications
You must be signed in to change notification settings - Fork 108
add EEN CORS allowlist #1684
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
base: master
Are you sure you want to change the base?
add EEN CORS allowlist #1684
Conversation
templates/nginx/galaxy-main.j2
Outdated
|
|
||
| # This is currently (2025-10) needed for the EOSC EU Node | ||
| location = /authnz/keycloak/login { | ||
| proxy_pass http://galaxy_upstream; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is galaxy_upstream correct? in other places we have just galaxy
| upstream galaxy { |
| proxy_pass http://galaxy; |
|
https://doc.traefik.io/traefik/reference/routing-configuration/http/middlewares/headers/ |
|
Regarding the NGINX part, I can not 100% say if this is correct by just looking at it, I would need to test it |
|
@mira-miracoli Björn came around this morning during the workshop and metioned that what's deployed on the server doesn't exactly match this PR, we'd have to look at what's actually deployed (and this PR would need to be updated). |
|
Updated the PR. What I don't know is how to add: to the http section of /etc/nginx/nginx.conf |
Don't merge yet, not clear if needed.
The EOSC EU Node wants to redirect users from the EU Node automatically, without login (SSO in the EOSC EU Node), back to Galaxy.
This might be needed. Thanks to Uwe for providing this snippet.
A review would be nice, so that I can merge and deploy it when it's needed.
See specifically the one comment in the code.