html-critical-webpack-plugin icon indicating copy to clipboard operation
html-critical-webpack-plugin copied to clipboard

Add support to files with cache busting suffix

Open abdoIO opened this issue 4 years ago • 2 comments

Files that look like main.css?t=123456 cause error since cache busting suffix is not supported. These changes added support to these kind of files.

abdoIO avatar Apr 30 '20 13:04 abdoIO

Thanks for the PR @abdoio 👍

Could you provide the error that you saw? Just curious and then all it can be a good reference for anyone else who comes across this issue.

thescientist13 avatar May 03 '20 18:05 thescientist13

@thescientist13 Thank you for your prompt response.

Here is the error log [ERROR] Error: css should not be empty [ERROR] at <node_modules/penthouse/lib/index.js>:220:33 [ERROR] at Generator.next (<anonymous>) [ERROR] at step (<node_modules/penthouse/lib/index.js>:19:191) [ERROR] at <node_modules/penthouse/lib/index.js>:19:437 [ERROR] at new Promise (<anonymous>) [ERROR] at <node_modules/penthouse/lib/index.js>:19:99 [ERROR] at Promise (<node_modules/penthouse/lib/index.js>:241:20) [ERROR] at new Promise (<anonymous>) [ERROR] at module.exports (<node_modules/penthouse/lib/index.js>:182:10) [ERROR] at <node_modules/critical/lib/core.js>:191:16 [ERROR] at tryCatcher (<node_modules/bluebird/js/release/util.js>:16:23) [ERROR] at Promise._settlePromiseFromHandler (<node_modules/bluebird/js/release/promise.js>:517:31) [ERROR] at Promise._settlePromise (<node_modules/bluebird/js/release/promise.js>:574:18) [ERROR] at Promise._settlePromise0 (<node_modules/bluebird/js/release/promise.js>:619:10) [ERROR] at Promise._settlePromises (<node_modules/bluebird/js/release/promise.js>:699:18) [ERROR] at Promise._fulfill (<node_modules/bluebird/js/release/promise.js>:643:18) [ERROR] at Promise._resolveCallback (<node_modules/bluebird/js/release/promise.js>:437:57) [ERROR] at Promise._settlePromiseFromHandler (<node_modules/bluebird/js/release/promise.js>:529:17) [ERROR] at Promise._settlePromise (<node_modules/bluebird/js/release/promise.js>:574:18) [ERROR] at Promise._settlePromise0 (<node_modules/bluebird/js/release/promise.js>:619:10) [ERROR] at Promise._settlePromises (<node_modules/bluebird/js/release/promise.js>:699:18) [ERROR] at Promise._fulfill (<node_modules/bluebird/js/release/promise.js>:643:18) [ERROR] at Promise._resolveCallback (<node_modules/bluebird/js/release/promise.js>:459:14) [ERROR] at ReductionPromiseArray._resolve (<node_modules/bluebird/js/release/reduce.js>:61:19) [ERROR] at ReductionPromiseArray._resolveEmptyArray (<node_modules/bluebird/js/release/reduce.js>:52:10) [ERROR] at ReductionPromiseArray.init (<node_modules/bluebird/js/release/promise_array.js>:71:18) [ERROR] at Promise._settlePromise (<node_modules/bluebird/js/release/promise.js>:571:21) [ERROR] at Promise._settlePromise0 (<node_modules/bluebird/js/release/promise.js>:619:10) [ERROR] at Promise._settlePromises (<node_modules/bluebird/js/release/promise.js>:699:18) [ERROR] at _drainQueueStep (<node_modules/bluebird/js/release/async.js>:138:12) [ERROR] at _drainQueue (<node_modules/bluebird/js/release/async.js>:131:9) [ERROR] at Async._drainQueues (<node_modules/bluebird/js/release/async.js>:147:5) [ERROR] at Immediate.Async.drainQueues (<node_modules/bluebird/js/release/async.js>:17:14) [ERROR] at runCallback (timers.js:705:18) [ERROR] at tryOnImmediate (timers.js:676:5) [ERROR] at processImmediate (timers.js:658:5)

abdoIO avatar May 04 '20 09:05 abdoIO