/incubator-pagespeed-mod

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rel=canonical links added for ipro resources #1222

Closed
jeffkaufman opened this Issue Dec 16, 2015 · 0 comments

Comments

Assignees

@jeffkaufman jeffkaufman

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@jeffkaufman
@jeffkaufman
Copy link
Contributor

jeffkaufman commented Dec 16, 2015

With 1.10 we add rel=canonical links to rewritten resources. This is applying too broadly, though,
and IPRO-optimized resources are getting it too:

$ curl -sS -D- localhost:8080/mod_pagespeed_example/images/Puzzle.jpg -o/dev/null
...
Link: <http://localhost:8080/mod_pagespeed_example/images/Puzzle.jpg>; rel="canonical"

@jeffkaufman jeffkaufman self-assigned this Dec 16, 2015

@jeffkaufman jeffkaufman referenced this issue Dec 16, 2015

Closed

Tracking bug: 1.10.33.0 issues #1216

@jeffkaufman jeffkaufman closed this in 61f4e96 Dec 17, 2015

jeffkaufman added a commit that referenced this issue Dec 17, 2015

@jeffkaufman
Don't include rel=canonical headers on IPRO resources. 
Fixes #1222
da843e8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment