Skip to content

Rename root#436

Open
halpomeranz wants to merge 71 commits into
tclahr:mainfrom
halpomeranz:rename-root
Open

Rename root#436
halpomeranz wants to merge 71 commits into
tclahr:mainfrom
halpomeranz:rename-root

Conversation

@halpomeranz
Copy link
Copy Markdown
Contributor

Renaming "[root]" to "collected_files" makes things easier in the uac code and also for analysts using the collected files.

I'm not wedded to "collected_files" if you want to change the name to something else. There are only a very few spots in the uac script needed to make the change.

This is apropos of Issue #435

ekt0-syn and others added 30 commits August 21, 2025 15:28
Detect running processes that inserted BPF filters in the Linux server
Update ss.yaml to show bpf filters
Add output and log filenames to the Azure Storage SAS URL.

Fixes tclahr#389
The output and log file names are now automatically appended to the URL provided in `--azure-storage-sas-url` ([tclahr#389](tclahr#389)). Consequently, the `--azure-storage-sas-url-log-file` option is no longer needed and has been removed.
feat: add statf tool for FreeBSD based systems
fix: parse special permissions in statx binary
Add an action to close stale pull requests older than 180 days.
artif: add binfmt_misc artifact
Resolved a bug that prevented proper artifact collection when the mountpoint of a mounted disk image included spaces or special characters.
tclahr and others added 26 commits January 17, 2026 10:46
Add one more parameter as command was added to _find_based_collector function.
feat: find collectors may include a command to use with xargs
artif: add additional possible persistence locations
Move artifact to a different artifact directory.
Add changelog.
Collect the SSH private key path only, and not the full key content.
Add System.map* to avml.yaml.
fix: look for systemd journal only in /var/log
artif: collect SSH public keys, test secret keys for null passphrases
Not only is the new name more descriptive, it's a lot less hassle in
both the "uac" code and also for later analysis.

Signed-off-by: Hal Pomeranz <hrpomeranz@gmail.com>
@halpomeranz
Copy link
Copy Markdown
Contributor Author

Unit tests will need to be updated to account for the new directory name

@tclahr
Copy link
Copy Markdown
Owner

tclahr commented Feb 18, 2026

I decided to use [root] with [] as a referente to FTK which also uses [root] to map root file systems.

The issue here is that some tools such as Cyber Triage, dissect... they might be using [root] as a reference to find and parse files.
If we decide to rename [root], I would say it should be done in a major release (v4).
I would use files as the new name as it matches with what we have inside artifacts directory.

@william-billaud
Copy link
Copy Markdown
Contributor

Regarding the dissect plugins it would be easy to change it. Let me know if you plan it one day (or I will probably see it).

But I'm agree, the '[root]' name is now part of the UAC 'API' and is probably used in a lot of automation and changing it would be better in a major release.

@halpomeranz
Copy link
Copy Markdown
Contributor Author

How much more code do I have to push in order for you to call it a major release? :-)

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.

5 participants