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

Resource double-encoded "cf.d41...7e.ce,od5...93,ostyle,,s,s.css" #29

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

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Enable mod_pagespeed on DreamHost VPS
2. Reload page
3. src attributes are messed up

What is the expected output? What do you see instead?

(brackets removed to avoid security filtering)
link rel=stylesheet media=screen type="text/css" 
href="http://mysite.com/css/cf.d41d8cd98f00b204e9800998ecf8427e.ce,ob256868a9324
 4a99784557e47350a6d0,ostyle,,s,s.css" 
(should be "/css/style.css")

script type="text/javascript" 
src="http://mysite.com/js/jm.802a2d00203636ecca34bd0154cb2842.jquery-1,o3,o1,omi
 n,l.js" 
(should be "/js/jquery.1.3.1.min.js")

script type="text/javascript" 
src="http://mysite.com/js/jm.b24e034c27e1ed8e45224bf36d7688b9.jquery,oform,l.js"

(should be "/js/jquery.form.js")

What version of the product are you using (please check X-Mod-Pagespeed
header)? not sure

On what operating system?
Linux version 2.6.33.7-vs2.3.0.36.30.4 (root@womb) (gcc version 4.3.2 (Debian 
4.3.2-1.1) )

Which version of Apache?
Apache/2.2.15

Which MPM?
unknown?

Please provide any additional information below, especially a URL or an
HTML file that exhibits the problem.
It's disabled right now.



Original issue reported on code.google.com by dojoh...@gmail.com on 5 Nov 2010 at 5:01

@GoogleCodeExporter
Copy link
Author

This is intentional. the "random characters" is a MD5 hash of the contents used 
to cache-extend the resource followed by a simple encoding of the original 
filenames.

Were the links broken for you? That would be a bug.

Please see examples on http://www.modpagespeed.com/ and look at the source of 
the before and after links for examples of how this is supposed to work.

Thanks,
-Shawn

Original comment by sligocki@google.com on 5 Nov 2010 at 6:02

@GoogleCodeExporter
Copy link
Author

Weird... I re-enabled it to make sure the files loaded and such, and it's 
working just fine. Well it's good to know that is expected. Thank you for the 
quick response!

Original comment by dojoh...@gmail.com on 5 Nov 2010 at 6:56

@GoogleCodeExporter
Copy link
Author

Yay

Original comment by sligocki@google.com on 5 Nov 2010 at 7:32

  • Changed state: Done

@GoogleCodeExporter
Copy link
Author

Ok I might have lied... sort of. The main page I was looking at works now, but 
another, on the same server, does not now that I've enabled this.

http://omg.stage.digdug.me/

The css file loads up blank.
http://omg.stage.digdug.me/css/cf.d41d8cd98f00b204e9800998ecf8427e.ce,od5e8ef952
74a54ebd45e0df007651893,ostyle,,s,s.css

versus

http://omg.stage.digdug.me/css/style.css

Original comment by dojoh...@gmail.com on 5 Nov 2010 at 10:52

@GoogleCodeExporter
Copy link
Author

Yep, that looks like a CSS minifier issue, I've attached the offending CSS and 
give it a look.

Original comment by sligocki@google.com on 5 Nov 2010 at 11:26

  • Added labels: Priority-High
  • Removed labels: Priority-Medium

Attachments:

@GoogleCodeExporter
Copy link
Author

Original comment by sligocki@google.com on 5 Nov 2010 at 11:26

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

Is that the same as mine? Or should I try replacing mine with that?

Original comment by dojoh...@gmail.com on 5 Nov 2010 at 11:33

@GoogleCodeExporter
Copy link
Author

I just attached your CSS file so that I can reference a fixed version in case 
your site's version changed.

Original comment by sligocki@google.com on 5 Nov 2010 at 11:39

@GoogleCodeExporter
Copy link
Author

seems to be working now.... no idea what changed (nothing in the style sheet 
that I know of). Might be some kind of initial caching issue? I dunno.

Original comment by dojoh...@gmail.com on 6 Nov 2010 at 8:20

@GoogleCodeExporter
Copy link
Author

I'm suspicious about the doubly-rewritten uri; it looks like we're cache 
extending, *then* minifying (we ought to do the reverse, causing cache 
extension to be a no-op).  Either the filters are strangely re-ordered, or 
we're encountering a really strange loopback problem during resource lookup.  
That might mean that you need to request the resource 2-3 times before you get 
correctly rewritten data back (and you may get a 404 in the mean time).  I'm 
hoping we can reproduce this locally and figure out what's going on and why.

Original comment by jmaes...@google.com on 7 Nov 2010 at 5:47

@GoogleCodeExporter
Copy link
Author

good luck. let me know if I can be any help

Original comment by dojoh...@gmail.com on 7 Nov 2010 at 5:51

@GoogleCodeExporter
Copy link
Author

One plausible cause for the doubly-rewritten URLs is if the mod_pagespeed 
output-filter winds up the Apache filter-chain more than once.

This could happen in a virtual hosting setup where mod_pagespeed is configured 
twice:

Instance 1:  enabled_filters=extend_cache
Instance 2:  enabled_filters=rewrite_css,extend_cache

This could account for URLs like those pasted above.  dojohnso, is this 
possible in your setup?


One thing we can do in our system is to scan the response-headers for 
X-Mod-Pagespeed and skip the second rewriting pass if the header is already 
present.

Original comment by jmara...@google.com on 9 Nov 2010 at 10:47

@GoogleCodeExporter
Copy link
Author

I tried the CSS file in the stand-alone CSS parser, and it seems to work fine, 
so I'm guessing the issue here is with running through mod_pagespeed twice.

Original comment by sligocki@google.com on 10 Nov 2010 at 10:30

  • Changed title: Resource double-encoded "cf.d41...7e.ce,od5...93,ostyle,,s,s.css"

@GoogleCodeExporter
Copy link
Author

I'm not sure about the setup. I'm on a DreamHost VPS, and they added the 
optional setting per domain for PageSpeed. Should I submit a ticket to them 
with this thread?

Original comment by dojoh...@gmail.com on 10 Nov 2010 at 10:34

@GoogleCodeExporter
Copy link
Author

Yes, that would be great, thanks.

Original comment by sligocki@google.com on 10 Nov 2010 at 11:01

@GoogleCodeExporter
Copy link
Author

ticket submitted to DreamHost

Original comment by dojoh...@gmail.com on 10 Nov 2010 at 11:12

@GoogleCodeExporter
Copy link
Author

the response from DH:

One our dev's checked this out and he mentioned "I'm pretty sure it's
just a bug in pagespeed. We just turn on the core filters plus
collapse_whitespace and remove_quotes, and I don't see a way for it to be
turned on twice."

I'm not worried about it at this point. I'm actually switching hosts for some 
of my sites that would use this, so I'm good here.

Original comment by dojoh...@gmail.com on 16 Nov 2010 at 6:41

@GoogleCodeExporter
Copy link
Author

We're curious if anyone is still experiencing this problem with recent versions 
of mod_pagespeed.  We have not been able to reproduce the issue, and will 
reopen it if we can get a reproducible test case on a recent release.

Original comment by jmaes...@google.com on 2 Dec 2010 at 4:28

  • Changed state: WontFix

@GoogleCodeExporter
Copy link
Author

I think I have the same issue.

http://ianstormtaylor.com/designs

Everything is showing up as those weird filenames, and it all works fine except 
for 3-4 of the images. (Open the developer console and see where it gives 
errors for a couple of them.)

Not sure what to do... been trying everything.

Thanks for any help.

Original comment by storm%st...@gtempaccount.com on 14 Dec 2010 at 8:56

@GoogleCodeExporter
Copy link
Author

Ian - Looks like you're running 0.9.10.1; is there any chance you could try 
0.9.11.5 (the current binary release on the site) to see if this resolves the 
problem?  We took a crack at this problem two weeks ago and I'd like to know if 
it's still causing trouble.

I've re-opened pending confirmation.

Original comment by jmaes...@google.com on 14 Dec 2010 at 2:59

  • Changed state: New

@GoogleCodeExporter
Copy link
Author

Ah ok. I'll talk to Dreamhost and see why they're serving a later version.

Thanks and I'll r back with an update.

Original comment by storm%st...@gtempaccount.com on 14 Dec 2010 at 4:51

@GoogleCodeExporter
Copy link
Author

This problem has been fixed for some time.

Original comment by jmara...@google.com on 21 Dec 2010 at 11:08

  • Changed state: Fixed
  • Added labels: release-note

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant