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

Tool installation fixes mainly #853

Merged
merged 3 commits into from
Mar 14, 2024
Merged

Tool installation fixes mainly #853

merged 3 commits into from
Mar 14, 2024

Conversation

kleozzy
Copy link
Contributor

@kleozzy kleozzy commented Mar 12, 2024

So I've updated brutespray , waymore, xnLinkFinder install with their new installation methods.
I have updated the instalaltion checks for the above.
I have updated the calling of the tools since they can now be called globally via path binary instead of using python.
I have updated the Post install message to include Github tokens and waymore config path.

kleozzy added 3 commits March 12, 2024 22:22
Changed to direct repo paths to resolve dependency issues
Update end of install message
Copy link
Owner

@six2dez six2dez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey mate, thanks for the pull request. I have a few key points to share:

  • brutespray installation, gives you the binary at brutespray/brutespray/main, check the difference below, make sure brutespray is accessible from the PATH or you need to specify the full route:
    image

  • Waymore instructions to install with pip are git+https://github.com/xnl-h4ck3r/waymore.git, so this is the command that should be included in requirements.txt

@kleozzy
Copy link
Contributor Author

kleozzy commented Mar 13, 2024

Hey man its 3 commits actually so the 2nd commit has the proper waymore and xnlinkfinder install url b007c08 also brutespray works globally if its installed via go -v install , so it works fine :)

@kleozzy
Copy link
Contributor Author

kleozzy commented Mar 13, 2024

Just like below, now that i look at it we might need to also change how its been called withing reconftw.sh since it doesnt need the path anymore
image

@six2dez six2dez merged commit ed8f58d into six2dez:dev Mar 14, 2024
@burper00 burper00 mentioned this pull request Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants