-
Notifications
You must be signed in to change notification settings - Fork 45
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
WARNING in webpack-subresource-integrity: Cannot determine asset for chunk scripts, computed="undefined", available=scripts.2b355891e0dd05f2cdd5.js. #104
Comments
Thanks, could you share your Webpack config as well, perhaps in a Gist? |
@jscheid Here is the content of
|
Thanks. Which version of this plugin ( |
I am not using it directly, I think its a part of Angular CLI |
What does this output?
or if you're using
|
|
Ok, you need to upgrade it. Try |
okay, sure, |
It don't upgrade like that, I think its a part of some sub dependency, and it is not getting upgrade like that from project root. |
Right, it's because angular depends on an older version of this plugin, which I believe they do because they disagree with #90. You could work around it by switching to yarn and specifying an override. The following might work but I haven't tested it: {
"resolutions": {
"webpack-subresource-integrity": "^1.3.2",
}
} I'm going to close this because the only thing I could do to help is revert #91 or maintain a separate branch, and I'd only do that after further discussion (on ticket #90). Please do let me know if the workaround helps, though. |
@jscheid simple resolution with yarn didn't work.
|
Thanks. The grep result you've posted doesn't conclusively show that it didn't work, I assume you've tried re-running the build and it ran into the same error again? |
Yes
On Wed, 6 Feb 2019 at 2:11 PM, Julian Scheid ***@***.***> wrote:
Thanks. The grep result you've posted doesn't conclusively show that it
didn't work, I assume you've tried re-running the build and it ran into the
same error again?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#104 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIdItOPeJJ61jR3bhi1cEcTBvSahnYprks5vKpw5gaJpZM4ahXOG>
.
--
Best Regards
Atif Saddique
|
@atifsaddique211f the upgrade works fine in my test, see https://gist.github.com/jscheid/29dee9816e89a27c71b54fd40719a70f If it still doesn't work for you I'm happy to take another look. I'll need the following from you:
|
I am also getting same warning on |
@vishalhulawale |
For those that are facing this due to angular, the issue is being sorted out at angular/angular-cli#9338 (essentially the above solution by @jscheid is not updating the dependencies recursively, so the cli package should fix the version in their dependency). |
The text was updated successfully, but these errors were encountered: