Skip to content
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

Added Nginx to serving arches, with config #327

Merged
merged 1 commit into from
Apr 10, 2023
Merged

Added Nginx to serving arches, with config #327

merged 1 commit into from
Apr 10, 2023

Conversation

ekansa
Copy link
Collaborator

@ekansa ekansa commented Mar 28, 2023

brief description of changes

This provides some instruction on using Gunicorn and Nginx, with an example Nginx config that describes how to compress css, javascript, etc.

issues addressed

#267

further comments

I am using more or less the same Nginx config in my Docker deployed Arches instance, so It seems to work. I'm not an Nginx guru, so I'm sure it could be better, but it's nice to provide a working starting point for people wanting to deploy Arches with Nginx.

You can view the update here;
https://arches.readthedocs.io/en/nginx_m/deployment/serving-arches-with-apache/#configure-nginx


This box must be checked

  • the PR branch was originally made from the base branch

This box should be checked

  • after these changes the docs build locally without error

This box should only be checked you intend to follow through on it (we can do it on our end too)

  • I will cherry-pick all commits in this PR into other branches that should have them after this PR is merged

@ekansa ekansa requested review from aarongundel and mradamcox March 28, 2023 21:55
Copy link
Member

@mradamcox mradamcox left a comment

Choose a reason for hiding this comment

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

Great! Good to have NGINX in here finally.

@ekansa ekansa merged commit 3d670a3 into master Apr 10, 2023
@ekansa ekansa deleted the nginx_m branch April 19, 2023 20:12
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