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

Version 0.2.0 #55

Merged
merged 20 commits into from
Sep 26, 2024
Merged

Version 0.2.0 #55

merged 20 commits into from
Sep 26, 2024

Conversation

FieryIceStickie
Copy link
Member

Merging #54 into this one since tests get changed again in this PR and I would rather changes get reviewed on the most modern copy.
Tests have been added (when they are fairly easy to create, certain schemas will require some work so don't have tests yet).

Deobfuscators are now modular and can just be added by adding files to the deobfuscators directory. This allows for a few things:

  • Versioning of deobfs is now possible
  • Adding new deobfuscators are streamlined (No need to change code in like 15 places, just add a new file and the code will pick it up automatically)
  • Deobfs can be lazy loaded, so if you're working on another deobf with broken code and need to use the program, it will run fine as long as you don't try to load the broken deobf
  • Specifying multiple deobfs to be ran on a single file is now possible (Currently the code will terminate on the first successful deobf, this is easy to change if desired)

Minor changes:

  • Changed -s/--soft to --show-expected
  • Added -s/--skip-scan to skip scanning of deobfs and just run them

FieryIceStickie and others added 19 commits August 19, 2024 23:07
Signed-off-by: stickie <[email protected]>
Signed-off-by: stickie <[email protected]>
…nce it's not available anymore)

Signed-off-by: stickie <[email protected]>
Signed-off-by: stickie <[email protected]>
Signed-off-by: stickie <[email protected]>
- Move current utils.py -> deobf_utils.py
- Move logging and color code in cli.py -> utils.py

Signed-off-by: stickie <[email protected]>
Signed-off-by: stickie <[email protected]>
Signed-off-by: stickie <[email protected]>
Signed-off-by: stickie <[email protected]>
Signed-off-by: stickie <[email protected]>
Copy link
Member

@import-pandas-as-numpy import-pandas-as-numpy left a comment

Choose a reason for hiding this comment

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

Approved with no blockers, but in the future let's reason about a way to specify including a specifier to install the package with or without test data. I worry that this will trip AV, and I don't feel great about arbitrarily including obfuscated code in our package.

@import-pandas-as-numpy import-pandas-as-numpy merged commit ffa1f66 into main Sep 26, 2024
4 checks passed
@import-pandas-as-numpy import-pandas-as-numpy deleted the major-refactor branch September 26, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants