Problem/Motivation

The file path handling in EmbeddedImageValidator is broken when used with files. My sites/default/files dir is a symlinked to /mnt/files/.

strpos fails because $this->fileSystem->realpath() resolved the symlink already.
getDrupalRootRelativePath removes the / from the resovled path.

I think its best to continue with the absolute path from fileSystem->realpath once it has been resolved.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 3554992-5.patch2.39 KBwebflo

Issue fork symfony_mailer_lite-3554992

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

webflo created an issue. See original summary.

webflo opened merge request !30

webflo closed merge request !30

webflo reopened merge request !30

webflo’s picture

webflo commented
Status: Active » Needs review
StatusFileSize
new 3554992-5.patch2.39 KB

Static patch for composer

  • zengenuity committed 85471e24 on 2.0.x authored by webflo 
    Issue #3554992 - EmbeddedImageValidator does not handle symlinks
zengenuity’s picture

zengenuity commented
Status: Needs review » Fixed

Merged. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.