-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Update payload.txt1 #541
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
base: master
Are you sure you want to change the base?
Update payload.txt1 #541
Conversation
DELAY 1000 | ||
CTRL-ALT t | ||
DELAY 2000 | ||
|
||
|
||
function remove_folder { |
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.
Is this meant to be inside a STRINGLN
block?
REM ########################################## | ||
|
||
TOKEN=your_dropbox_token |
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.
This is not the proper way to set a variable within Duckyscript
DEFINE #TOKEN your_dropbox_token
REM Requirements: | ||
REM - Internet Connection | ||
REM - Discord Webhook | ||
|
||
mkdir -p $DIR |
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.
This is not proper Duckyscript syntax, is this meant to be executed on the target machine?
Also, as this is a change to anther users payload, it requires review from the original author @aleff-github |
I don't have a way to test the payload at this time, so (@Duck43233) perhaps including a description of the new features in the Pull Request would help me understand what it's about. In general, I usually agree with the new versions proposed by users, but given @hak5peaks' suggestions, I would prefer to wait for an official and corrected version. |
stuffmlb