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

/etc/nginx/conf.d/default.conf should be named 50-default.conf to help config file ordering #2593

Open
p12tic opened this issue Mar 1, 2025 · 0 comments

Comments

@p12tic
Copy link
Contributor

p12tic commented Mar 1, 2025

Currently nginx-proxy puts its configuration into a file named default.conf which is included via include /etc/nginx/conf.d/*.conf; directive. The nginx configuration is order dependent, thus if users rely on anything in default.conf being ordered before or after their config file, they must choose the name for the config file accordingly. Currently the name must be something that comes either before or after default.conf in alphabetical order. This is confusing.

A simple fix for this would be to rename the generated config file from default.conf to have some number prepended, such as 50-default.conf.

Please let me know if this issue is something for which a PR would be accepted. I would then go prepare one.

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

No branches or pull requests

1 participant