Redirect /index.php/* to /* in Drupal Nginx images

[seanhamlin]

See https://www.drupal.org/forum/support/post-installation/2020-04-06/solved-drupal-8-is-adding-indexphp-in-url for background. In short, Drupal 8 has valid routes for /index.php/blah where the friendly URL is /blah. This only serves to confuse Google, and the end user.

There is best practice in https://www.nginx.com/resources/wiki/start/topics/recipes/drupal/ to fix this with

    # Enforce clean URLs
    # Removes index.php from urls like www.example.com/index.php/my-page --> www.example.com/my-page
    # Could be done with 301 for permanent or other redirect codes.
    if ($request_uri ~* "^(.*/)index\.php/(.*)") {
        return 307 $1$2;
    }

We should add this to the base images.

[Schnitzel]

mhh we already do something like this:
https://github.com/amazeeio/lagoon/blob/8128a4db750a61fcda62e6d5206be6c4865506e9/images/nginx-drupal/drupal.conf#L12-L14

I guess this could conflict?

[seanhamlin]

I guess we could just update the existing rule with the recommended rule from Nginx. There was a customer that had found a bunch of URLs that were prefixed by /index.php/ in Google (so clearly the current redirect is not perfect).

[tobybellwood]

Closing as no further recent conversation - please reopen if this use case or issue still exists.