Skip to content
Public Ticket #14495

Issue on URL sanitizing

Conversation thread

PenciDesign replied

Staff Reply

Hi,

Ok, I will send this issue for our developers to check it and we will add the fix in the next update of AMP pluginsmile.pngNo worries about this.

Best Regards,

PenciDesign

JohnDF replied

Customer Reply

Hi there,

Yes. I updated themes and all plugins, including AMP plugin, before submitting the ticket.

PenciDesign replied

Staff Reply

Hi,

Have you using latest version of AMP plugin ( version 3.4 )? If not, let's update by do following steps:

1/ Go to download page and re-download "All files and documentation" package. After downloaded it, extract the download package > open the folder you received > open plugins folder > and you can see penci-soledad-amp.zip file for latest version of AMP plugin.

2/ Go to plugins > All Plugins > deactivate & delete old AMP plugin.

After that, upload latest version of AMP plugin you got on step 1.

And try it again.

If this issue still appears, let me know and I will send for our developers to check it and add the fix in the next update of AMP pluginsmile.png

Best Regards,

PenciDesign

JohnDF started the conversation

Conversation Start

If a tag page starts with "amp", the following class doesn't sanitize it the right way:  includes/sanitizers/class-amp-link-sanitizer.php

For example: https://knitmuch.com/tag/amphora/

The amphtml tag was being displayed as:

<link rel="amphtml" href="https://knitmuch.com/tag/amphora/" />

Instead of:

<link rel="amphtml" href="https://knitmuch.com/amp/tag/amphora/" />

I applied a temporary fix to includes/amp-frontend-actions.php, to prevent errors and warnings on Google Search Console.

Want to join this conversation or access attachments? Sign in first.