10000 Null in sources content skips reading of file · Issue #147 · webpack-contrib/source-map-loader · GitHub
[go: up one dir, main page]
Skip to content

Null in sources content skips reading of file #147

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
Closed
ilgonmic opened this issue May 17, 2021 · 3 comments
Closed

Null in sources content skips reading of file #147

ilgonmic opened this issue May 17, 2021 · 3 comments

Comments

@ilgonmic
Copy link
Contributor
  • Operating System: macOS BigSur 11.2.3
  • Node Version: 16.0.0
  • NPM Version: 7.10.0
  • webpack Version: 5.37.0
  • source-map-loader Version: 2.0.1

Expected Behavior

null in sources content considered as sources should be retrieved by name (http://sourcemaps.info/spec.html)

Actual Behavior

null in sources content considered as skip reading of file

Code

{"version":3,"file":"untitled.js","sources":["kotlin-wrappers-kotlin-react-dom-jsLegacy.js","../../../../../src/main/kotlin/client.kt","../../../../../src/main/kotlin/welcome.kt"],"sourcesContent":[null,null,null], "names":[], "mappings":";"}

How Do We Reproduce?

Use source map with null in sourcesContent but having source file in file system.
@alexander-akait
Copy link
Member

I see, thanks, I will fix it

@ilgonmic
Copy link
Contributor Author

I have fix in branch, I am testing it. So feel free to use it
https://github.com/ilgonmic/source-map-loader/tree/fix-null-source-content

8000

@ilgonmic ilgonmic mentioned this issue May 17, 2021
Merged
6 tasks
@alexander-akait
Copy link
Member

Fixed #148

ilgonmic added a commit to JetBrains/kotlin that referenced this issue Jun 7, 2021
@ilgonmic
[Gradle, JS] Actualize versions of source-map-loader because of bug
e827617 
webpack-contrib/source-map-loader#147

^KT-46608 fixed
Alefas pushed a commit to JetBrains/kotlin that referenced this issue Jun 8, 2021
@ilgonmic
[Gradle, JS] Actualize versions of source-map-loader because of bug
4c3e922 
webpack-contrib/source-map-loader#147

^KT-46608 fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alexander-akait @ilgonmic
0