Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use nginx as reverse proxy in production #50

Merged
merged 12 commits into from
Jan 31, 2022
Merged

Conversation

bjubes
Copy link
Member

@bjubes bjubes commented Mar 14, 2021

Uses nginx with static file caching instead of traefik

to test, .envs/.production/.secrets should resemble:

# PostgreSQL
# ------------------------------------------------------------------------------
POSTGRES_HOST=postgres
POSTGRES_PORT=5432
POSTGRES_DB=osler
POSTGRES_USER=BnnIJhssshZrnURWgfjnnEXZRMzhNZCx
POSTGRES_PASSWORD=JzQ6eHA47iiEzURQo1xJ2VPeGpRY81edS1UpuQc82KP5bb7T8t6qR7ANFTRK5bxI


# Django
# ------------------------------------------------------------------------------
DJANGO_SECRET_KEY=PbQjPuCXmpX4dHJITSI2nSJy6lzivrHkyxIZJkAnowUsEzsWkucovzd75yz8BqVH

Copy link
Contributor

@justinrporter justinrporter left a comment

Choose a reason for hiding this comment

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

Awesome work!

@bjubes
Copy link
Member Author

bjubes commented Mar 19, 2021

i think it makes sense to leave this PR as is until #51 is merged, then ill incorporate those changes into this one

@justinrporter
Copy link
Contributor

sgtm!

@bjubes
Copy link
Member Author

bjubes commented Aug 17, 2021

@justinrporter I combined the changes from #58 into here, as well as combined the documentation. Let me know what you think. Also, I don't see the docs page from http://localhost:7000/ or http://localhost:7000/getting-started/ does the list not autogenerate?

Also: I modified /.envs/.production/.umkc to to keep DJANGO_ALLOWED_HOSTS=osler.umkc.edu, please check this is correct as it isn't present in #58

@bjubes bjubes requested review from justinrporter and wwick August 17, 2021 23:23
@wwick wwick merged commit c90bc7d into llemr-conspiracy:master Jan 31, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants