Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Part of #189
This is the first pass at adding a basic artifact storage to the framework. Changes are:
exploit.StoreArtifact
that will store an artifact in theoutput
flag location and/or in the databaseThings needed before undraft:
conf.SetStringFlag("output")
will change behavior seems good. I honestly considered making this a "reserved" flag somehow and doing something similar to Allow for manual triggering of C2 startup #300 so that "output artifact data" becomes a config toggle explicitly. I think the magic config flag isn't ideal and could be a footgun. Another benefit of doing it Allow for manual triggering of C2 startup #300 style would be that the exploit internals could add to-details
if artifact collection is expected.