Unify webserver log path retrieval

[qmadev]

Related to #264.

The idea here is to use the _log_paths() _get_paths() function to retrieve the resolved log paths for the various webservers that we support. Future webservers should implement this function to automatically collect the resolved log files with Acquire. They also need to be added in the Acquire collection module.

[Schamper]

Can you use _get_paths instead? That's the currently defined standardized API for this:

dissect.target/dissect/target/plugin.py

Line 504 in 06990b8

def _get_paths(self) -> Iterator[Path]:

See #1082 for more details.

[qmadev]

Can you use _get_paths instead? That's the currently defined standardized API for this:

dissect.target/dissect/target/plugin.py

Line 504 in 06990b8

def _get_paths(self) -> Iterator[Path]:

See #1082 for more details.

Fixed!

[codecov]

Codecov Report

❌ Patch coverage is 50.00000% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.99%. Comparing base (3c8e8f6) to head (99608e4).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
dissect/target/plugins/apps/webserver/apache.py	40.00%	6 Missing ⚠️
dissect/target/plugin.py	40.00%	3 Missing ⚠️
dissect/target/plugins/apps/webserver/iis.py	40.00%	3 Missing ⚠️
dissect/target/plugins/apps/webserver/nginx.py	57.14%	3 Missing ⚠️
dissect/target/plugins/apps/webserver/caddy.py	71.42%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1287      +/-   ##
==========================================
- Coverage   81.02%   80.99%   -0.04%     
==========================================
  Files         376      376              
  Lines       33594    33627      +33     
==========================================
+ Hits        27221    27237      +16     
- Misses       6373     6390      +17     

Flag	Coverage Δ
unittests	80.99% <50.00%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[codspeed-hq]

CodSpeed Performance Report

Merging #1287 will not alter performance

_{Comparing qmadev:fix/webserver_collection (99608e4) with main (3c8e8f6)}

Summary

✅ 9 untouched

[qmadev]

Fixed the linter, checks should be good now @twiggler

Dismissed.