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

webserver.apache : Support Include/IncludeOptionnal directive in apache configurations files #1013

Open
william-billaud opened this issue Feb 6, 2025 · 2 comments

Comments

@william-billaud
Copy link

Currently, webserver.apache parser try to read configuration from apache in order to add possible access/error logs path (

for config in self.DEFAULT_CONFIG_PATHS:

If vhost configuration made in dedicated files using the Include or IncludeOptional they are ignored.

It could be possible to:

  • Retrieve ServerRoot (used as base fore relative path)
  • Retrieve included path using Include(Optionnal) Directive
  • Parse identified vhost configuration files included in order to extract potential access/error logs path.

I can make a PR if you want.

@JSCU-CNI
Copy link
Contributor

I think PR #980 takes care of this. Could you confirm @william-billaud?

@william-billaud
Copy link
Author

Exactly thanks, I hadn't found it because I only search among issue and not open PR...

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

2 participants