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

extending data collection on solaris #229

Merged
merged 7 commits into from
May 16, 2024
Merged

Conversation

sec-hbaer
Copy link
Contributor

While using UAC on Solaris, I found some data was not covered yet:

  • script files that are run by services (using the service management facility of solaris)
  • config files for services (using the service management facility of solaris)
  • config files of the link-editor (ld)
  • information on installed packages
    • there is an artifact for this already, but it doesnt cover what we normally consider as packages
  • information files of installed packages (containing metadata of the package, changes to the system and checksums+metadata of files that are added by the package)
  • files around logins to the system

I also noticed that the bodyfile with a depth of 6 does not cover all files that are normally on a solaris system. E. g. the pkg content files have a depth of seven on the file system.
I did not propose a change to the depth limit, as Im unsure as to why it was introduced in the beginning.

I added artifact collection files and modified some existing ones to cover such data. I tested it successfully on a Solaris version 11.4

If needed, I can provide a uac collection archive with the new artifacts / changes for review :)

@tclahr
Copy link
Owner

tclahr commented May 16, 2024

Awesome!
Concerning the max_depth limit in bodyfile, that was added as bodyfile creation was taking too long to run on some systems such as macos. I changed the code completely on uac v3, so that limitation will be removed when v3 is released (in the next months).

@tclahr tclahr merged commit c2304a6 into tclahr:develop May 16, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants