Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for offsite image processing #392

Closed
iansvo opened this issue Jun 22, 2022 · 0 comments
Closed

Support for offsite image processing #392

iansvo opened this issue Jun 22, 2022 · 0 comments
Labels
[Type] Enhancement A suggestion for improvement of an existing feature

Comments

@iansvo
Copy link

iansvo commented Jun 22, 2022

Feature Description

Native image processing inside WordPress adds a ton of convenience, but it does inevitably cause more strain on the server. The effect grows more pronounced when processing larger amounts of images, using a smaller (or underpowered) webserver.

While I am not familiar with the implementation in this repo directly, I would imagine this might take the form of some kind of filter and webhook combo so the custom processing mechanism could "listen" for new queue items from the site. There would need to be some similar return mechanism, perhaps another webhook that is just standardized and only in use if the processing happens offsite? Just a few ideas.

Example processing flow

  1. User uploads attachment to media library
  2. Media library determines file is eligible for conversion
  3. Attachment Object is sent to the webhook destination
  4. Once processing completes, new POST made to the site to send back the image and associated attachment meta for update

Why?

While smaller sites are unlikely to need a thing like this, I imagine the burden for a larger publication/blog could be substantial, especially if there are many users making uploads all at once. It might be worth doing some kind of stress test to determine the positive impact of a feature like this, although I'm not sure how that might be done organically.

It's worth noting this is a concern that others have echo'd on the core proposal as well: https://make.wordpress.org/core/2022/03/28/enabling-webp-by-default/#comment-42843

@mxbclang mxbclang added [Type] Enhancement A suggestion for improvement of an existing feature [Focus] Images labels Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Enhancement A suggestion for improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

4 participants