javascript-obfuscator / javascript-obfuscator/webpack-obfuscator

process variable is undefined when use DefinePlugin

Open
#123 2 comments 2 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
928
Forks
92
PR merge metrics
No merged PRs in 30d

Description

I wrote about this bug in [another issue](https://github.com/javascript-obfuscator/javascript-obfuscator/issues/863), but it is already closed, it would probably be more correct to make a separate issue in a more suitable repository for this.

I made a [minimal example](https://github.com/Katarn/obfuscator-process-bug).

Just run `npm ci && npm run build`. After that, in the `dist` directory, you will see the `main.js` file. Open it in a code editor and you will see something like:

```js
....shift())}}(n),process[r(309)][r(311)]&&console[r(302)](r(311))})();...
```

Of course, the `process` variable is `undefined`. Moreover, it should not be in the code, it should either be replaced with `true` (according to [this line](https://github.com/Katarn/obfuscator-process-bug/blob/master/webpack.config.js#L22)), or, with "smart optimization", the `if (...)` check block should not exist at all, since it will always be executed.

Then you can uncomment these [two lines](https://github.com/Katarn/obfuscator-process-bug/blob/5edcd56c5f2f18b6068e8459f7be075fc9691bb5/webpack.config.js#L14-L15) (not necessarily at the same time, you can uncomment them separately) and see how the code is generated in the `main.js` file.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the minimal example's webpack.config.js, especially the DefinePlugin configuration around lines 14-15 and 22, then run npm ci && npm run build. Inspect dist/main.js with the options enabled and compare whether process is replaced or optimized away as expected.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript, webpack
Domain
build-system
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.