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

eventlog_creds Module #452

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

eventlog_creds Module #452

wants to merge 7 commits into from

Conversation

lodos2005
Copy link
Contributor

@lodos2005 lodos2005 commented Oct 13, 2024

Screenshot 2024-10-14 at 01 03 53 Screenshot 2024-10-14 at 01 03 20

I have added a new module that extracts user credentials from Windows event logs. The main focus is on monitoring logs, such as Sysmon or Windows Security logs Event ID 4688. If computers are monitored using Windows Event Forwarding (WEF) or SIEM solutions, executed commands and their arguments can be stored in the event log. I designed this module to find credentials in those logs and extract them.

add new module

Signed-off-by: Hakan Yavuz <lodos05@gmail.com>
Signed-off-by: Hakan Yavuz <lodos05@gmail.com>
fix for lint

Signed-off-by: Hakan Yavuz <lodos05@gmail.com>
@NeffIsBack
Copy link
Contributor

Thanks for the PR :)

@Dfte
Copy link
Contributor

Dfte commented Oct 14, 2024

Hey dude!

Really nice PR! Is there a way you can retrieve the event log files without running additional execute() operation ?

@lodos2005
Copy link
Contributor Author

lodos2005 commented Oct 14, 2024

Hey dude!

Really nice PR! Is there a way you can retrieve the event log files without running additional execute() operation ?

Hm, maybe we use even/even6, i will look that.

Signed-off-by: Hakan Yavuz <lodos05@gmail.com>
add rpc method, 

Signed-off-by: Hakan Yavuz <lodos05@gmail.com>
@lodos2005
Copy link
Contributor Author

well, i added rpc method also. rpc call is a bit slower on large logs but I think it is more stealth.
Screenshot 2024-10-15 at 01 18 52
Screenshot 2024-10-15 at 01 18 08
I don't know which one should be the default.

fix for linter, spaces

Signed-off-by: Hakan Yavuz <lodos05@gmail.com>
@Dfte
Copy link
Contributor

Dfte commented Oct 15, 2024

Amazing!!! To be honest I'd rather have RPC being the default one as it will improve stealthness and allow attacker to dump credentials without rising alerts because of the execute :)

Signed-off-by: Hakan Yavuz <lodos05@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants