Drupal 10 / Nginx - Compiled assets broken with too many redirects on Drupal 10.1

ericgsmith commented 1 year ago

When using the drupal10 recipe with nginx and Drupal 10.1 I am unable to access CSS and JS when preprocessing / aggregation is enabled.

Example config:

recipe: drupal10
config:
  via: nginx

This is the same issue that was closed against Drupal core as being down to outdated nginx config https://www.drupal.org/project/drupal/issues/3368769

has

  location @rewrite {
      rewrite ^/(.*)$ /index.php?q=$1;
  }

Where the nginx example config has

  location @rewrite {
      #rewrite ^/(.*)$ /index.php?q=$1; # For Drupal <= 6
      rewrite ^ /index.php; # For Drupal >= 7
  }

I have tested this change locally using config override on the appservice and it fixed the issue I was experiencing.

MR incoming

reynoldsalec commented 1 year ago

Merged the PR and should come out in the next Lando v3.20.x release

reynoldsalec commented 1 year ago

This should now be fixed in Lando 3.20.4, please re-open if it's still a problem @ericgsmith, thanks for the contribution!

lando / drupal