Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

ModPagespeedLoadFromFile should strip query string params #357

Closed
GoogleCodeExporter opened this issue Apr 6, 2015 · 5 comments
Closed

ModPagespeedLoadFromFile should strip query string params #357

GoogleCodeExporter opened this issue Apr 6, 2015 · 5 comments
Labels
auto-migrated Priority-Medium release-note Type-Enhancement

Comments

@GoogleCodeExporter
Copy link 

I have ModPagespeedLoadFromFile set to look in my directory for the appropriate 
files instead of requesting them from Apache. However, it seems that the query 
string isn't being stripped before modpagespeed tries to find the file.


[...] [error] [mod_pagespeed 0.10.19.4-1209 @20786] 
/home/[...]/public_html/wp-content/plugins/wp-cycle/jquery.cycle.all.min.js?ver=
3.2.1:0: open input file (code=2 No such file or directory)


Just to be clear, that file does exist.

Original issue reported on code.google.com by amat...@gmail.com on 6 Dec 2011 at 11:53
@GoogleCodeExporter
Copy link
Author 

Yes, we do not strip query strings. Perhaps we should.

But this seems a bit odd, why are you using a query string for a static file? I 
would assume query strings are only used for dynamically generated files (is 
this being used for cache busting?)

Original comment by sligocki@google.com on 7 Dec 2011 at 4:27

@GoogleCodeExporter
Copy link
Author 

Yeah, the version query string is inserted automatically (and unconditionally) 
by the WordPress CMS. It's used for cache busting so that new versions of the 
assets are served up as they release new versions of the platform. Take a look 
at the script / style URLs in the source of http://ebayinkblog.com to see many 
more examples of such URLs.

Without the ability to strip query strings, ModPagespeedLoadFromFile is sadly 
useless on WordPress.

Original comment by amat...@gmail.com on 7 Dec 2011 at 5:57

@GoogleCodeExporter
Copy link
Author 

Makes sense and I can't think of any reason not to strip query strings. Looking 
into this.

Original comment by sligocki@google.com on 7 Dec 2011 at 6:05

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author 

This issue was closed by revision r1231.

Original comment by sligocki@google.com on 7 Dec 2011 at 9:57

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Original comment by sligocki@google.com on 7 Dec 2011 at 9:59

  • Added labels: release-note

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
auto-migrated Priority-Medium release-note Type-Enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GoogleCodeExporter