Problem/Motivation

When using the https://www.drupal.org/project/imagemagick toolkit with Gutenberg.

When multiple media entities are loaded by the editor at a time (think up to 100s). It can lead to lots of processes being spawned when the (with some not being able to finish because the server locks up).

This is an issue that needs to be addressed within that module. However, we should also cache the responses to reduce the chances of this happening.

Proposed resolution

Cache the /editor/media/load/{file} and /editor/media/load-media/{media} routes.

Remaining tasks

Provide cache.

User interface changes

N/A

CommentFileSizeAuthor
#4 3557437-gutenberg-cache-entity-lookup--applies-on-3334899.diff1.59 KBcodebymikey

Issue fork gutenberg-3557437

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

codebymikey created an issue. See original summary.

codebymikey opened merge request !228

codebymikey’s picture

codebymikey commented
Status: Active » Needs review
codebymikey’s picture

codebymikey commented
StatusFileSize
new 3557437-gutenberg-cache-entity-lookup--applies-on-3334899.diff1.59 KB

Attached a patch that applies on top of the work done in #3334899: Validate Media entities created through the MediaTypePersistenceManager

  • codebymikey committed b0adcb07 on 4.0.x 
    feat: #3557437 Cache media and file lookup controller

By: codebymikey
codebymikey’s picture

codebymikey commented
Version: 8.x-2.x-dev » 4.0.x-dev
Status: Needs review » Fixed

Merged to 4.0.x since that's where all the momentum for new features/fixes should be directed to.

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.