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

Remove All Base64 References from GTFOBins Database #189

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

Conversation

calebstewart
Copy link
Owner

Description of Changes

The base64 stream type in gtfobins.json is no longer valid, but there were entries left behind using it. This pull request is meant to remove all remnants of that stream type due to the updates to LinuxFile which now supports raw data without any encoding.

I've tested the changes so far with the Peak Hill room, but I need to go through and test each possible GTFObin transfer method to make sure it's working properly. Before merging this, I want to write proper unit tests that will test each one explicitly to make sure they all work.

Fixes #173.

Major Changes Implemented:

  • Remove GTFOBins base64 stream type

Pre-Merge Tasks

  • Formatted all modified files w/ python-black
  • Sorted imports for modified files w/ isort
  • Ran flake8 on repo, and fixed any new problems w/ modified files
  • Ran pytest test cases
  • Added brief summary of updates to CHANGELOG (under [Unreleased])

For issues with pre-merge tasks, see CONTRIBUTING.md

@calebstewart calebstewart mentioned this pull request Sep 19, 2021
5 tasks
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.

Empty /etc/passwd after auto escalation
1 participant