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

Question: Generation of source map files #179

Open
andycook71 opened this issue Jun 22, 2021 · 13 comments
Open

Question: Generation of source map files #179

andycook71 opened this issue Jun 22, 2021 · 13 comments

Comments

@andycook71
Copy link

Is there a way for WebOptimizer to allow for serving of source maps, (*.js.map)? Currently source source maps are resulting in a 404 for me.

I see that that CodeBundlingSettings has a reference to NUglify's SymbolsMap, but can't find any example of using it.

@benjamin-stern
Copy link
Contributor

benjamin-stern commented Nov 28, 2021

Any updates on this?

I think the bigger issue, is that there is no way to bundle it with the library so that it would reference it from memory or bundle the map along with the generated output js.

trullock/NUglify#11

@rferreiraperez
Copy link

Is there a plan to add this functionality?

@jwbats
Copy link

jwbats commented Oct 30, 2022

I just ran into this, as well.

No maps support for inlined scripts.

@xilonoide
Copy link

Ugh! no updates to this.

@QuinnDamerell
Copy link

+1, this would be really useful!

@MrCodeB2
Copy link

+1

@b9chris
Copy link
Contributor

b9chris commented Jun 3, 2024

Is there still interest in this? I'm considering adding this feature in a Fork, and if it would be welcome, adding it to the core project however you guys prefer to do things. Don't know that process.

@jwbats
Copy link

jwbats commented Jun 3, 2024

@b9chris, I would still be interested.

@xilonoide
Copy link

Is there still interest in this? I'm considering adding this feature in a Fork, and if it would be welcome, adding it to the core project however you guys prefer to do things. Don't know that process.

yes! I'm!

@b9chris
Copy link
Contributor

b9chris commented Jun 12, 2024

It's working on my Fork.

Because I'm also building Nuglify from source, it seems to be failing the AppVeyor test (AppVeyor can't find Nuglfy so it fails to build). I'll see if I can rebuild the Pull Request with Nuglify from Nuget...

#310

@QuinnDamerell
Copy link

This is amazing! My main use case for this is uploading the source maps to New Relic or Sentry so the services can parse and categorize the errors better. Thanks for your work!

@b9chris
Copy link
Contributor

b9chris commented Jun 14, 2024

This should be ready for merge, here's the Pull Request:

#311

@b9chris
Copy link
Contributor

b9chris commented Jul 18, 2024

(This feature has been merged and this Issue should probably be closed as resolved)

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

No branches or pull requests

8 participants