Double entity encoding issue

barryhunter

unread,

Mar 14, 2012, 12:06:53 PM3/14/12

to page-speed-se...@googlegroups.com

I'm testing running my site though PSS - so far only for me using a hosts file.

Getting entity encoding issues - on some pages only.

On page

http://www.geograph.org.uk/discuss/index.php?&action=vthread&forum=2&topic=14660

has a source...

<td width="100%" class=caption3><a title="Forum Listing" href="http://www.geograph.org/discuss/index.php?&">Discuss</a> / <a title="Topic listing for General Discussion" href="http://www.geograph.org.uk/discuss/index.php?&action=vtopic&forum=2">General Discussion</a> / A refrain that divided the generations</td>

but when run though PSS, gets converted to

<td width="100%" class=caption3><a title="Forum Listing" href="index.php?&">Discuss</a> / <a title="Topic listing for General Discussion" href="index.php?&action=vtopic&amp;forum=2">General Discussion</a> / A refrain that divided the generations</td>

which has &amp; - which of course doesnt work.

Now yes, I know its odd that we have a encoded ampersand entity in there & and a unecoded one & - but the browser copes with this.

So will be fixing the forum code (when I can find it - the forum software is third party) - but it seems like PSS shouldnt be breaking urls - that otherwise work.

Where there is only encoded <a href="search.php?i=29433514&page=2">2</a> or only unencoded <a href="index.php?&"> , it works fine.

barryhunter

unread,

Mar 14, 2012, 12:19:03 PM3/14/12

to page-speed-se...@googlegroups.com

Doh, realised that page requires signin - and will change when I fix it (think done it already)

Put a copy of relevent code here http://www.geograph.org.uk/stuff/pss-amp.php to test.

Compare

$ GET http://www.geograph.org.uk/stuff/pss-amp.php

<a title="Forum Listing" href="http://www.geograph.org.uk/discuss/index.php?&">Discuss</a> / <a title="Topic listing for General Discussion" href="http://www.geograph.org.uk/discuss/index.php?&action=vtopic&forum=2">General Discussion</a> / A refrain that divided the generations

$ GET http://209.85.147.121/stuff/pss-amp.php -H'Host: www.geograph.org.uk'

<a title="Forum Listing" href="/discuss/index.php?&">Discuss</a> / <a title="Topic listing for General Discussion" href="/discuss/index.php?&action=vtopic&amp;forum=2">General Discussion</a> / A refrain that divided the generations

Satyanarayana Manyam

unread,

Mar 14, 2012, 1:11:11 PM3/14/12

to barryb...@gmail.com, page-speed-se...@googlegroups.com

Hi,

Thanks for putting sample pages for us to test. We will look into this and get back to you soon.