Join the Soledad Facebook Users Group here
for Share, assistance, discussion, and Updates related to Soledad WordPress Theme.

If you can't create a new ticket - you can send us an email via our contact form and we will get back to you.

Okay
  Public Ticket #2573327
Issue on URL sanitizing
Closed

Comments

  •  2
    JohnDF started the conversation

    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.

  •  2,699
    PenciDesign replied

    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

  •  2
    JohnDF replied

    Hi there,


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

  •  2,699
    PenciDesign replied

    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