Repository navigation
target node change class name #531
Description
Class Name should change when calling static methods
Expected Behavior
- As the documentation says
Currently the output of browser and node is identical.
But there is a difference between them.
- When a class name change it should change the class name in static method calls
Current Behavior
- difference between the target node and browser
- when calling static methods class name should also change
Steps to Reproduce
Code exmple:
class A {
static async one(){
}
static async two(){
await A.one()
}
}
When using target=Brwoser:
class A {
static async ['one']() {
}
static async ['two']() {
await A['one']();
}
}
When using target=node:
class _0xd91f5a {
static async ['one']() {
}
static async ['two']() {
await A['one']();
}
}
When running with a target node you get error because A is not defined
Your Environment
- Obfuscator version used: 0.24.0
- Node version used: 12.4
Activity
sanex3339 commentedon Jan 27, 2020
Will try to fix soon
sanex3339 commentedon Jan 27, 2020
renameGlobalsbreaking this code toosanex3339 commentedon Jan 27, 2020
@LiadDrori currently Node target should affect only how identifiers are renamed
sanex3339 commentedon Jan 27, 2020
Should sanitize class scopes like this:
sanex3339 commentedon Jan 27, 2020
After this pr, here should be no difference between Node and Browser targets in terms of variable names.
sanex3339 commentedon Jan 27, 2020
Released as
0.24.1please try itsanex3339 commentedon Jan 27, 2020
Nice!