8000 docs: Add a note about configuration ordering in /etc/nginx/conf.d by p12tic · Pull Request #2607 · nginx-proxy/nginx-proxy · GitHub
[go: up one dir, main page]

Skip to content

docs: Add a note about configuration ordering in /etc/nginx/conf.d #2607

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

Merged
merged 1 commit into from
Apr 11, 2025

Conversation

p12tic
Copy link
Contributor
@p12tic p12tic commented Apr 8, 2025

Users may not be aware that the filename chosen to add extra configuration in /etc/nginx/conf.d may interfere with nginx-proxy. nginx-proxy stores its configuration at /etc/nginx/conf.d/default.conf. Any files that come before default.conf in alphabetical order will be applied before default.conf and vice-versa.

This is confusing, because the documentation does not specify how exactly nginx-proxy interacts with nginx.

This can be remedied by a simple note alerting the users of the importance of filename choices.

Fixes #2593.

Users may not be aware that the filename chosen to add extra
configuration in /etc/nginx/conf.d may interfere with nginx-proxy.
nginx-proxy stores its configuration at /etc/nginx/conf.d/default.conf.
Any files that come before default.conf in alphabetical order will be
applied before default.conf and vice-versa.

This is confusing, because the documentation does not specify how
exactly nginx-proxy interacts with nginx.

This can be remedied by a simple note alerting the users of the
importance of filename choices.
@buchdag buchdag merged commit 4534241 into nginx-proxy:main Apr 11, 2025
@p12tic p12tic deleted the doc-config-file-ordering branch April 11, 2025 16:07
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.

/etc/nginx/conf.d/default.conf should be named 50-default.conf to help config file ordering
2 participants
0