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

Add new Nginx config to base image #918

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add new Nginx config to base image #918

wants to merge 1 commit into from

Conversation

nbianca
Copy link
Member

@nbianca nbianca commented Jan 17, 2025

This commit moves the Nginx config from discourse/discourse repository
and updates it with several 'include' statements that serve as extension
outlets.

This change was made to place the config closer to where it is used and
implements an alternative to the "find & replace" hacks.

It is unused at this moment, but all templates and samples will be
updated in a future commit.

This commit moves the Nginx config from discourse/discourse repository
and updates it with several 'include' statements that serve as extension
outlets.

This change was made to place the config closer to where it is used and
implements an alternative to the "find & replace" hacks.

It is unused at this moment, but all templates and samples will be
updated in a future commit.
@nbianca nbianca changed the title Add Nginx configuration to base image Add new Nginx config to base image Jan 17, 2025
@tgxworld
Copy link
Contributor

@nbianca Do confirm that the existing web template will override the nginx conf file that has been added to the base image:

- "cp $home/config/nginx.sample.conf /etc/nginx/conf.d/discourse.conf"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants