Skip to content

AAI-281 Forward proxy headers so static files for DB admin can be loaded#44

Merged
marius-mather merged 2 commits into
mainfrom
fix-deployment-db-admin
Aug 1, 2025
Merged

AAI-281 Forward proxy headers so static files for DB admin can be loaded#44
marius-mather merged 2 commits into
mainfrom
fix-deployment-db-admin

Conversation

@marius-mather

@marius-mather marius-mather commented Jul 31, 2025

Copy link
Copy Markdown
Collaborator

Description

AAI-281: need to forward headers from the AWS load balancer so that the JS/CSS files the database admin uses can be loaded properly. See https://aminalaee.github.io/sqladmin/cookbook/deployment_with_https/

Note these options are potentially insecure for a web service exposed directly to the internet, but our service is only exposed to the AWS load balancer.

Changes

  • Update docker run command to include --proxy-headers and --forwarded-allow-ips options

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have added unit / integration tests that prove my fix is effective or that my feature works
  • I have run all tests locally and they pass
  • I have updated the documentation (if applicable)

How to Test Manually (if necessary)

Build the docker image locally and run it:

docker build -t aai_backend .
docker run -p 8000:8000 aai_backend

@amandazhuyilan amandazhuyilan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the change!

@marius-mather marius-mather merged commit 2c31efe into main Aug 1, 2025
2 checks passed
@amandazhuyilan amandazhuyilan deleted the fix-deployment-db-admin branch June 19, 2026 03:21
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.

2 participants