-
Notifications
You must be signed in to change notification settings - Fork 404
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
Add new SMB module to download Screenshots created by Snipping Tool #368
Conversation
Nice addition !! 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your PR! Great work and really readable code :)
2 small changes that need to be done, i will test it the upcomding days. Then we are ready for mergin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, a few things came up during testing.
When you finished coding please lint the code using ruff. You can find an example how this is done is in our PR template:
https://github.com/Pennyw0rth/NetExec/blob/main/.github/PULL_REQUEST_TEMPLATE.md
ae35552
to
6418de3
Compare
6418de3
to
af512f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey,
By default, the (newer) Windows Snipping Tool saves copies of screenshots in
%USERPROFILE%/Pictures/Screenshots
, even if you save the file in another directory. The settings for this behavior are managed locally, and apparently it is not possible to disable this feature via Group Policy (GPO).Reference: Nathan McNulty on Twitter
With the help of some existing modules, I created this module to automatically download all screenshots from either all users or specific users (similar to #335.).