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.

Sign up for GitHub

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

Jump to bottom

Fixes issue #1350 by properly identifying obfuscated text #2105

Open

Donovan55 wants to merge 2 commits into beautifier:main from Donovan55:patch-1

Donovan55 commented Oct 2, 2022 •

edited by bitwiseman

Description

Source branch in your fork has meaningful name (not main)

Fixes #1350

Before Merge Checklist

These items can be completed after PR is created.

(Check any items that are not applicable (NA) for this PR)

JavaScript implementation
Python implementation (NA if HTML beautifier)
Added Tests to data file(s)
Added command-line option(s) (NA if
README.md documents new feature/option(s)


          Fixes issue beautifier#1350 by properly identifying obfuscated text

da09339

bitwiseman requested changes

View reviewed changes

Member

bitwiseman left a comment

Please add test cases showing how this change works to the run_tests method in this same file.

Also update https://github.com/beautify-web/js-beautify/blob/49cd14de2f83e4a2c4c4ed5a3a04077efc17924c/python/jsbeautifier/unpackers/javascriptobfuscator.py#L48


          Merge branch 'main' into patch-1

8a8ceba

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment