Hello,
Indeed, that’s the intended behaviour.
The ignore list exists only for compatibility when there is a merging conflict and it’s not meant to leave the scripts completely alone.
Query strings should always be removed from static assets, as they do nothing to the script itself, other than to bypass cache.
If you are not running PHP inside the .js file, this changes is good for you and part of the optimization of fvm.
But I’ll consider adding an option not to remove the query string of ignored files.
Will add it to my roadmap.
Thanks for reporting,
Raul
Thread Starter
Fabian
(@gravitas)
Hi Raul,
thanks for considering.
In our case query strings are indeed intended behavior and part of our versioning / client cache busting strategy.
Removing query strings from static resources is a recommendation on pingdom.com tests, that’s why fvm uses file names instead of query strings.
What I can suggest for now, if you have control over the file query, is to change its name if you need cache invalidation. Something like filename-v2.js for example.
But I’ll look into this in the future.
Thanks again
