Skip to content
This repository has been archived by the owner on Apr 26, 2021. It is now read-only.

Added creation/deletion of registry keys to static HTML report summary #411

Closed
wants to merge 6 commits into from
Closed

Added creation/deletion of registry keys to static HTML report summary #411

wants to merge 6 commits into from

Conversation

depaolism
Copy link

These changes allow for more detailed information about the registry to be displayed in the web gui. It does not differentiate between key addition and modification and so treats all writes the same. Writes have green text and are prefixed with 'w+', deletions have red text and are prefixed with '-'. An example is below:

stry

Ben Schmoker and others added 5 commits November 12, 2014 12:42
… data structure. Enhanced now returns a dictionary of lists (events & regevents) instead of just a list (events).
… will display the value that was written. Additionally, writes to the registry will be in green text and prefixed with a 'w+' and deletes will be prefixed with a '-'.
@botherder botherder changed the title Feature/registry Added creation/deletion of registry keys to static HTML report summary Dec 1, 2014
@depaolism
Copy link
Author

Were there any changes you would like to see with how the creation/deletions are displayed. I was also thinking about adding similar changes for files and directories if that is something people would be interested in.

@botherder
Copy link
Member

Generally i think it's a fine pull request. I'm generally just overwhelmed with the current status of behavior.py. I would really like to see that tidied up (cc @Thorsten-Sick and @jbremer).

@jekil jekil added the Feature label Jan 31, 2015
@depaolism
Copy link
Author

Just checking on the status of this pull request. Is there something that I could do the help clean up behavior.py?

@gtback
Copy link
Contributor

gtback commented Mar 4, 2015

Is there a chance this will make it into the 1.2 release?

@botherder botherder added this to the 1.3 milestone Apr 3, 2015
@botherder botherder self-assigned this Apr 3, 2015
@botherder
Copy link
Member

I need to work on it.

Firstly, whenever the results structure changes, it causes compatibility issues. Secondly, the HTML report is pretty much abandoned, so this should be ported to the Django interface.
Lastly, as it is replacing the Summary, at that point I'd rather replace it completely for the other sections as well.

@depaolism
Copy link
Author

Screenshot of changes made by most recent commit.
files_summary

@jbremer
Copy link
Member

jbremer commented Jun 4, 2015

This functionality has been integrated as part of the monitor branch. Both the json report and the (deprecated) HTML report now show a summary with all these things in there, sorted by the type of event (file written, regkey created, etc). Integration for Django is still pending, but this is not the goal of this pull request anyway. So, closing this PR.

@jbremer jbremer closed this Jun 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants