-
-
Notifications
You must be signed in to change notification settings - Fork 68
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 wordlist symlinks, fzf-rules aliase and hashcat rules #394
base: dev
Are you sure you want to change the base?
Conversation
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.
thank you for the PR! small changes and we should be good to go
Hey @Tednoob17, Can you be compliant with folder path ? |
No problem, give me 5 minutes. |
Only last commit can be considered |
@ShutdownRepo, maybe we can move |
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.
Thx @Tednoob17
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.
Here are some changes following @qu35t-code's review and comments
Also do you mind changing the seclist install from git -C /opt clone --single-branch --branch master --depth 1 https://github.com/danielmiessler/SecLists.git seclists
to git -C /opt/lists clone --single-branch --branch master --depth 1 https://github.com/danielmiessler/SecLists.git seclists
Sorry for all those change requests, I didn't think the PR would question so many previous things 😅 but it's a good thing I guess
add-to-list "onelistforall,https://github.com/six2dez/OneListForAll,Rockyou for web fuzzing" | ||
} | ||
|
||
function install_rules(){ |
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.
We probably don't want to unzip the files here for two reasons
- it gets a bit heavy (72M)
- hashcat already support on-the-fly loading of compressed wordlists (https://github.com/hashcat/hashcat/blob/ba2746228a40ebeed53a187dfa54fc6154234481/docs/changes.txt#L14), I guess it works with rules as well
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.
Okay, i remove all gunzip, but with add-test-command
i keep it for checking zip files? @ShutdownRepo
sources/assets/shells/aliases.d/fzf
Outdated
@@ -1,4 +1,5 @@ | |||
alias fzf-wordlists='find /opt/rockyou.txt /opt/seclists /usr/share/wordlists /usr/share/wfuzz /usr/share/dirb -type f | fzf' |
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.
/opt/seclists
can be removed here since seclists will be in /opt/lists
(@qu35t-code was right, it's a good idea)
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.
Here i remove /opt/rockyou.txt
because now rockyou.txt
is in /opt/lists/rockyou.txt
and since i already added /opt/lists
/opt/rockyou.txt
is no longer relevant
sources/install/package_wordlists.sh
Outdated
mkdir -p /usr/share/wordlists | ||
ln -v -s /opt/seclists /usr/share/seclists | ||
ln -v -s /opt/seclists /usr/share/wordlists/seclists |
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.
these can be removed, I don't even know why they were created in the first place to be honest
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.
Yes, here i modify the following lines :
47 cd /opt/seclists || exit
by
cd /opt/lists/seclists || exit
And
50 add-test-command "[[ -f '/usr/share/wordlists/rockyou.txt' ]]"
51 add-test-command "[[ -d '/opt/seclists/Discovery/' ]]"
by
add-test-command "[[ -f '/opt/lists/rockyou.txt' ]]"
add-test-command "[[ -d '/opt/lists/seclists/Discovery/' ]]"
Can you add wordlists from assetnote ? https://wordlists.assetnote.io/ |
Hi @qu35t-code |
I added any function in packages_wordlists who add rules for hashcat, i implement fzf-rules aliase and create symlinks of directory contained a wordlists in /opt/lists .