-
Notifications
You must be signed in to change notification settings - Fork 1
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
Support relative includes #25
Comments
Hi there, Thanks for catching this, you make a good point. I didn't consider relative paths yet, and it seems that Nginx defaults to prefixing relative paths with
I'll see if I can fix this some time in the future, seems straightforward enough. |
I've checked, it's relative to root config: |
Thanks for the report. Should be fixed and tested in the latest release https://github.com/AlexanderGrooff/nginx-static-analysis/releases/tag/0.3.0 |
For example I have in nginx.conf line
include mime.types;
.Then with full path:
But if I start it with relative parameter it works:
(cd /etc/nginx;nginx-static-analysis --file nginx.conf)
.I think relative paths should be parsed against path of root config.
The text was updated successfully, but these errors were encountered: