-
Notifications
You must be signed in to change notification settings - Fork 642
Add new SMB module to download Screenshots created by Snipping Tool #368
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
Conversation
|
Nice addition !! 👍 |
NeffIsBack
left a comment
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
NeffIsBack
left a comment
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
NeffIsBack
left a comment
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.).