Image styles fails with absolute path in $settings['file_public_path']

Thank you, I took a look at the MR and left a couple of comments, see above. Also, the MR should fixed to point against 10.1.x to match the issue version.

VladimirAus → closed merge request !3387

Use 9.5 MR for Drupal 10.0.

Is this testing relative paths that resolve outside of the Drupal webroot? For example, I've been testing a setup where Drupal is installed in a subdirectory on the server, using $settings['file_public_path'] = '../public';. So a Drupal page would be https://my.server/path/to/drupal/web/node/1 and an uploaded image would be https://my.server/path/to/drupal/public/image01.png. That doesn't work with image styles, so I tried setting file_public_base_url to 'https://my.server/path/to/drupal/public'. That has many of the same symptoms described here.

Is adding the clean URL redirect lines to the public file folder .htaccess file going to be a requirement in any case? In order to do things like process the virtual URLs of image styles, for instance.

VladimirAus → changed the visibility of the branch 10.1.x to hidden.

VladimirAus → changed the visibility of the branch 2754273-10.1.x-image-styles-fails to hidden.

VladimirAus → changed the visibility of the branch 11.0.x to hidden.

Reroll the patch

Can MR be updated to 11.x please

have not reviewed for tests yet

is there a MR for Drupal 10.3?

@SocialNicheGuru current MR's diff is working for 10.3.

@smustgrave done

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Should be good now and pass phpcs

Thanks! Appears to still need tests though.