johnbenedict

Anyone? Any ideas?

Hi @westonruter — finally able to look into this for you. No we are not doing any rewriting of $_SERVER['REQUEST_URI']

I have no idea. I cannot find anywhere in our codebase where this is changed.

We are however using home_url() quite a bit — and we are always adding that trailing slash — so something like this: home_url( "/something/")

I did just test the output of home_url( '/' ) — and the trailing slash IS there. So it should be working in your function…

Hey @westonruter — after going over this with our developer — we’re not doing anything to home_url. No filter, manipulation, etc.

In order to fix the issue we were having — we made a simple update to amp_get_current_url():

This:
$url = preg_replace( ‘#(^https?://[^/]+)/.*#‘, ‘$1’, home_url( ‘/’ ) );

To this:
$url = rtrim(preg_replace( ‘#(^https?://[^/]+)/.*#‘, ‘$1’, home_url( ‘/’ ) ), ‘/’) . ‘/’;

But to keep from breaking the plugin — we actually went this route in footer.php:

$html = ob_get_clean();
$html = preg_replace("!(domain\.com)([a-z0-9])!i", '\1/\2', $html);
echo $html;

I hope this helps. Thank you for being so responsive and willing to help.

• This reply was modified 4 years, 9 months ago by johnbenedict.

After talking to our dev — we’re doing a good amount of customization > output buffering + link re-writing for our CDN. I probably should have discussed this with him first. So not sure it’d be all that helpful.

In amp_get_current_url() we’ve found a simple change that fixed our issue:

Change this:
$url = preg_replace( ‘#(^https?://[^/]+)/.*#‘, ‘$1’, home_url( ‘/’ ) );

To this:
$url = rtrim(preg_replace( ‘#(^https?://[^/]+)/.*#‘, ‘$1’, home_url( ‘/’ ) ), ‘/’) . ‘/’;

Ok — that is very likely what is happening. I do see some uses of home_url doing something like this: home_url('/something')

I will look into fixing this within our own setup.

Thank you so much for you help troubleshooting this.

Not sure I want to post all the Site Health info here…

WordPress is 5.3.1 (right now)
AMP Plugin is 1.5.3 and is in Transitional Mode

Home URL: https://www.campsitephotos.com
Site URL: https://www.campsitephotos.com/wp

• This reply was modified 4 years, 10 months ago by johnbenedict.

Yes. Like this in functions.php:

remove_action( 'wp_head', 'wp_generator' ); // remove wordpress version

• This reply was modified 4 years, 10 months ago by johnbenedict.

https://www.campsitephotos.com/

Not using multisite.

Install is standard.

AMP is active on Blog and Campground Pages right now:

https://www.campsitephotos.com/blog/camping-news/covid-19-campground-reopening-update/

https://www.campsitephotos.com/campground/ca/doheny-state-beach/

• This reply was modified 4 years, 10 months ago by johnbenedict.
• This reply was modified 4 years, 10 months ago by johnbenedict.

Thank you @westonruter! I really appreciate the fast response.

I’ll take the less ergonomic solution for now. Looking forward to the update.

{BUMP}

No one with any other ideas?

Ok – setups that work are:

mySQL: 5.2.6
Server API: Apache 2.0 Handler

Setup that does not work:

mySQL: 5.2.17
Server API: CGI/FastCGI

Not sure what other info would be needed to troubleshoot this?

Glad I’m not alone.

I actually have multiple sites that utilize WP Galleries – all on the latest version of WP.

All work just fine – except for this one – which is on a different host.

I’m wondering if it has something to do with mySQL or a PHP Config specific to this host???

Trying the .htaccess security fixes I found around Google didn’t help…

Also – I CAN edit Single Photos once they have been added. Something is up with Galleries.