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

sourceMap not working when used as a loader? #111

Open
Andy9822 opened this issue Feb 25, 2021 · 2 comments · May be fixed by #165
Open

sourceMap not working when used as a loader? #111

Andy9822 opened this issue Feb 25, 2021 · 2 comments · May be fixed by #165

Comments

@Andy9822
Copy link

Hi,

I've been using webpack-obfuscator for a couple of months and this week decided to enable sourceMap due to unreadable stacktrace being generated at our Sentry. So, it would be nice to have this sourceMap generation I thought... But it's not working.

I've been reading the issues and it seems you didn't have support for it a few months ago, it was also mentioned in the README.MD, but now it's supposed to be working. I upgraded the package version to its latest but couldn't make it work.

I'm using webpack-obfuscator as a loader, exactly as it is described in the README.MD and it works perfectly for everything else but when enabled sourceMap it just doesnt't generate anything, even with or without custom sourceMapFileName property.

@Andy9822 Andy9822 changed the title sourceMap not working when used as a loader sourceMap not working when used as a loader? Feb 25, 2021
@desk467
Copy link

desk467 commented Mar 1, 2021

I had done a test with javascript-obfuscator a while ago and I realized that it only generates the sourceMap when it runs via CLI. When executed as a webpack loader or as a webpack plugin, the settings for sourceMap seems to be ignored. Is this the expected behavior ?

@sanex3339
Copy link
Member

sanex3339 commented May 17, 2021

The loader does not support source maps. PR welcomed.
Btw, the plugin should support source maps after this PR: #104

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

Successfully merging a pull request may close this issue.

3 participants