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

Patch ducky logger #411

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Kr1ss-XD
Copy link

Watched the recent YouTube video and noticed some minor possible improvements to the payload.

fix shell history removal

The `rm -f` command was called without a target argument, since the
"HISTFILE" variable is unset before. This is fixed by this patch.
don't start systemd units twice

The `systemctl enable` command already starts the service/timer unit
given as an argument when the `--now` flag is included.
@dallaswinger dallaswinger added the requires review from author community contributed payload requires review or clarification from the original author label Jan 2, 2024
@dallaswinger
Copy link
Member

@drapl0n your review would be appreciated

@drapl0n
Copy link
Contributor

drapl0n commented Jan 4, 2024

Please merge the changes. Thanks for the PR @Kr1ss-XD

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires review from author community contributed payload requires review or clarification from the original author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants