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

Responder URL and Version Update for Quickcreds #49

Open
wants to merge 2 commits into
base: gh-pages
Choose a base branch
from

Conversation

w33ts
Copy link

@w33ts w33ts commented Feb 27, 2024

I updated Responder to work correctly out of the box and put the latest version onboard. I also tested it to ensure that it would still grab credentials, which was successful.

Huge shout out to dark_pyrro on the forums.

I updated Responder to work correctly out of the box and put the latest version onboard. I also tested it to ensure that it would still grab credentials, which was successful.

Huge shout out to dark_pyrro on the forums.
@w33ts w33ts changed the title Responder URL and Version Update Responder URL and Version Update for Quickcreds Feb 27, 2024
@w33ts
Copy link
Author

w33ts commented Feb 27, 2024

CleanShot 2024-02-27 at 08 26 02

@w33ts w33ts marked this pull request as draft February 27, 2024 22:06
@w33ts
Copy link
Author

w33ts commented Feb 27, 2024

Making some adjustments, as I don't believe I have tested this correctly. (Thanks dark_pyrro!)

This changes the version to 3.0.6.0, which works with Python 2. Huge shout out to dark_pyrro for sharing their knowledge and catching this. I've also adjusted the unpacking and handling based on their suggestion, in addition to removing the .exe deletion, as it no longer ships with the .exe. 

To validate functionality, I deleted the Responder folder on the SD card, the quickcreds loot folder, and ensured the /etc/turtle/modules/quickcreds file was up to date with this commit and confirmed that I received creds.
@w33ts
Copy link
Author

w33ts commented Feb 28, 2024

Creds folder after validation.

CleanShot 2024-02-27 at 19 38 52

@w33ts w33ts marked this pull request as ready for review February 28, 2024 01:55
@dallaswinger
Copy link
Member

review discussion took place on discord-- tldr; "LGTM"

merge pending one last test, lmk when this is ready to be merged @w33ts

thanks a ton!

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.

None yet

2 participants