Skip to content

Add CloudFront rewrites function, test, and deployment workflow#3390

Merged
natefoo merged 1 commit intogalaxyproject:masterfrom
natefoo:cloudfront-rewrites
Oct 10, 2025
Merged

Add CloudFront rewrites function, test, and deployment workflow#3390
natefoo merged 1 commit intogalaxyproject:masterfrom
natefoo:cloudfront-rewrites

Conversation

@natefoo
Copy link
Member

@natefoo natefoo commented Oct 10, 2025

The hub is currently served via nginx on an EC2 instance proxying the Hub S3 bucket. I'm moving this to CloudFront but needed a way to implement the nginx-based rewrites, which is now done via this function. I only added a couple testing rewrites for now and will follow up with the full set if everything works.

@natefoo
Copy link
Member Author

natefoo commented Oct 10, 2025

I don't really have a solution for this:

    location /events/gcc2021/training/ {
        proxy_http_version     1.1;
        proxy_set_header       Host shiltemann.github.io;
        proxy_pass             https://shiltemann.github.io/gcc2021-training/;
    }

Other than to add a redirect for it.

@natefoo natefoo merged commit 301a392 into galaxyproject:master Oct 10, 2025
4 checks passed
- name: Configure AWS Credentials
uses: aws-actions/configure-aws-credentials@v4
with:
aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any chance you can update this to use GitHub OIDC instead of maintaining these long lived keys?

https://github.com/aws-actions/configure-aws-credentials?tab=readme-ov-file#oidc-configuration :)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well hey Justin, nice to see you here. 🤯 😄

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

Successfully merging this pull request may close these issues.

3 participants