[report] --skip-files is ignored for dirs/symlinks created by Archive class

[pmoravec]

#3767 is incomplete in the following use case (artificial but simple to understand):

• have a plugin collecting /lib/alsa/init/info where /lib is a symlink to /usr/lib
• call sos report --skip-files=/usr/*

Then, the /lib/alsa/init/info files hould not be collected since whole /usr should be skipped. BUT the directory and symlink to it is created within https://github.com/sosreport/sos/blob/main/sos/archive.py#L254-L256 / https://github.com/sosreport/sos/blob/main/sos/archive.py#L258-L260 .

To prevent this, we must guard those two calls by Plugin._is_skipped_path(absdest) condition.

How to implement it in a gentle way? Options:

• Archive's methods add_file, check_path and _make_leading_paths to have extra argument list of skip paths, and the class will duplicate ``Plugin._is_skipped_path` method - I dont like passing potentially long lists, neither code duplication
• Archive's methods to pass just Plugin reference as an optional argument, and if present, Plugin._is_skipped_path(absdest) is called as the guard. But I don't like having references to plugins in Archive.
• the return value from Archive (https://github.com/sosreport/sos/blob/main/sos/report/plugins/__init__.py#L1532) is evaluated against _is_skipped_path, and if True, then..? remove the file/directory/..?

Some other / better idea, how to prevent creating symlinks to forbidden path this way?