web-infra-dev / rspack

The fast Rust-based web bundler with webpack-compatible API 🦀️
https://rspack.dev
MIT License
10.04k stars 573 forks source link

fix: set the correct asset_info when extracting css #8454

Closed inottn closed 4 days ago

inottn commented 5 days ago

Summary

fix #8431

The default asset_info is returned here: https://github.com/web-infra-dev/rspack/blob/9112e35f97eeac430691e15c0305763fec56e890/crates/rspack_plugin_extract_css/src/plugin.rs#L415

Therefore, the contenthash cannot be obtained here, leading to an unexpected termination of the subsequent process: https://github.com/web-infra-dev/rspack/blob/9112e35f97eeac430691e15c0305763fec56e890/crates/rspack_plugin_real_content_hash/src/lib.rs#L73-L75

Checklist

netlify[bot] commented 5 days ago

Deploy Preview for rspack canceled.

Built without sensitive environment variables

Name Link
Latest commit 4690eaed0ca12d801a1a5c15ee71af65a4b954fb
Latest deploy log https://app.netlify.com/sites/rspack/deploys/6739f6cc3ca6990008205de2
JSerFeng commented 4 days ago

Thanks !