Move from /amp to ?amp

Resolved fajferson
(@fajferson)

6 years, 6 months ago
Hello!

I have a problem with amp url.

Url with /amp have a problem with amp validation:
https://techunbox.pl/fortnite-wrescie-w-sklepie-play-czy-to-koniec-walki-epica-z-google-play-store/amp
https://search.google.com/test/amp?utm_source=gws&utm_medium=onebox&utm_campaign=suit&url=https%3A%2F%2Ftechunbox.pl%2Ffortnite-wrescie-w-sklepie-play-czy-to-koniec-walki-epica-z-google-play-store%2Famp

but ?amp works great
https://techunbox.pl/fortnite-wrescie-w-sklepie-play-czy-to-koniec-walki-epica-z-google-play-store?amp
https://search.google.com/test/amp?utm_source=gws&utm_medium=onebox&utm_campaign=suit&url=https%3A%2F%2Ftechunbox.pl%2Ffortnite-wrescie-w-sklepie-play-czy-to-koniec-walki-epica-z-google-play-store%3Famp

Why there is a problem with /amp?

I’m using amp with reader mode.

Link rel amphtl use /amp
<link rel="amphtml" href="https://techunbox.pl/fortnite-wrescie-w-sklepie-play-czy-to-koniec-walki-epica-z-google-play-store/amp">

Is there any way to change /amp to ?amp in link rel=”amphtml” to get validation valid or properly valid /amp?
- This topic was modified 6 years, 6 months ago by fajferson.
- This topic was modified 6 years, 6 months ago by fajferson.
The page I need help with: [log in to see the link]

Viewing 8 replies - 1 through 8 (of 8 total)

Plugin Author Weston Ruter
(@westonruter)

6 years, 6 months ago

Interesting. I’m seeing that too.

It appears your theme has an amp directory that contains the Reader mode templates, is that right?

Do you also have any add_theme_support('amp'…) in the functions.php?

Thread Starter fajferson
(@fajferson)

6 years, 6 months ago

Yes, my theme has amp directory.

There is no add_theme_support('amp'…) in my theme or child theme functions.php

Plugin Author Weston Ruter
(@westonruter)

6 years, 6 months ago

On the /amp/ version I see this at the footer:



Whereas on the ?amp version I do not.

This seems to indicate that the Cache Enabler plugin is causing the problem here, as when it runs it breaks AMP.

There was a support topic previously about Cache Enabler causing problems with AMP: https://wordpress.org/support/topic/lot-of-amp-error-in-the-google-searsh-console/#post-12627972

If you deactivate Cache Enabler does this fix the problem?

Plugin Author Weston Ruter
(@westonruter)

6 years, 6 months ago

A workaround for the Cache Enabler plugin would be to populate the Cache Exclusion field for “Regexp matching page paths that should not be cached.“ with:

/(\?amp(=.*))|\/amp\/?(\?.*)?$/

However, this will only work in Transitional mode and Reader mode.

Plugin Author Weston Ruter
(@westonruter)

6 years, 6 months ago

I’ve reported the issue regarding Cache Enabler incompatibility: https://github.com/keycdn/cache-enabler/issues/78
Thread Starter fajferson
(@fajferson)

6 years, 6 months ago
Yes, I’m using Cache Enabler plugin.

I’ve added /(\?amp(=.*))|\/amp\/?(\?.*)?$/ in Cache Enabler exclusion and now i have valid AMP page on /amp.

I’ll fallow this issue on github 🙂

Thanks for your help!
- This reply was modified 6 years, 6 months ago by fajferson.
- This reply was modified 6 years, 6 months ago by fajferson.
- This reply was modified 6 years, 6 months ago by fajferson.
Plugin Author Weston Ruter
(@westonruter)

6 years, 6 months ago

Great. We’d appreciate a plugin review!

Thread Starter fajferson
(@fajferson)

6 years, 6 months ago

Thats right 🙂

Viewing 8 replies - 1 through 8 (of 8 total)

The topic ‘Move from /amp to ?amp’ is closed to new replies.