Closed egibs closed 2 months ago
This looks good.
The next improvement that I can envision is for us to be able to present the original paths instead of the temp file path in the output. I'll open a separate issue for that.
Looks like a merge conflict, @egibs can you resolve so that we can get this merged?
Looks like a merge conflict, @egibs can you resolve so that we can get this merged?
Done!
Relates to: #173
Previously, only single archives were supported. This PR adds support for any number of archive files encountered during a recursive scan.
Since
recursiveScan
became unwieldy with the changes introduced in this PR, I split out the archive and file handling into separate functions.To handle archives within directories, the
processArchive
function recursively scans the new temporary directory created via thearchive
function.Example --
make test
still passes as expected with these changes and scanning single binaries and archives works as well.cc: @tstromberg @vaikas