[wp-trac] [WordPress Trac] #29201: File versioning should not use query strings, but rename the filename to allow caching

WordPress Trac noreply at wordpress.org
Sat Aug 23 03:02:57 UTC 2014


#29201: File versioning should not use query strings, but rename the filename to
allow caching
---------------------------+----------------------
 Reporter:  benoitchantre  |       Owner:
     Type:  enhancement    |      Status:  closed
 Priority:  normal         |   Milestone:
Component:  Script Loader  |     Version:  3.9.1
 Severity:  normal         |  Resolution:  wontfix
 Keywords:                 |     Focuses:
---------------------------+----------------------
Changes (by dd32):

 * status:  new => closed
 * resolution:   => wontfix
 * milestone:  Awaiting Review =>


Comment:

 Unfortunately this isn't something we can really implement.
 - It's up to Proxy admins to have correct caching rules, Squid has
 recommended caching "dynamic" url's since 2.7
 - A proxy config which deny's caching url's with ? will simply fall back
 to browser-based caching
 - WordPress can't really do this without adding extra rewrite rules
 - WordPress core doesn't load CSS/JS files directly, rather they go
 through a dynamic php script loader, so even if we did add rewrite rules
 and all that, it's still not going to be viable as we'd have a dynamic url
 in 99% of cases
 - Plugins/Themes can already do this if they pass the correct arguments to
 wp_register_script(), and rename the file with each release
 - The web has grown up greatly since 2008, unless currently core was
 significantly broke by overzealous caching (which it isn't) then I think
 we should rely on modern caching software to do things "right"

 I'm marking this as wontfix for now, you can still comment if you feel
 strongly about the issue.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/29201#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list