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

Critical issue in version 2.7.6 when used with S3 Uploads plugin #245

Closed
1 task done
leogermani opened this issue Nov 30, 2023 · 2 comments · Fixed by #247
Closed
1 task done

Critical issue in version 2.7.6 when used with S3 Uploads plugin #245

leogermani opened this issue Nov 30, 2023 · 2 comments · Fixed by #247
Assignees
Labels
type:bug Something isn’t working.
Milestone

Comments

@leogermani
Copy link

Describe the bug

We are having a severe issue with the latest released version (2.7.6) when used along a cloud storage plugin (in our case, we use S3-Uploads plugin). The result is the site becomes unresponsive.

The issue seems to have been introduced in #216.

From a quick look, the issues seems to be that the external S3 URL is returned here, and thus the replacement done here will never work.

This apparently causes the site to keep creating new image sizes forever.

This is a guess about the cause, not confirmed.

It would be awesome if you could have a look at this, because we are unable to update the plugin in many of our sites.

Thanks!

Steps to Reproduce

Screenshots, screen recording, code snippet

No response

Environment information

No response

WordPress information

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@faisal-alvi
Copy link
Member

@leogermani fix #247 (reverting #216) is merged into the develop branch, can you please confirm the issue you mentioned is fixed?

@faisal-alvi faisal-alvi reopened this Dec 8, 2023
@github-project-automation github-project-automation bot moved this from Merged to In Progress in Open Source Practice Dec 8, 2023
@faisal-alvi faisal-alvi added the needs:feedback This requires feedback to determine next steps. label Dec 8, 2023
@dkotter dkotter closed this as completed Dec 12, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Merged in Open Source Practice Dec 12, 2023
@leogermani
Copy link
Author

Hi @faisal-alvi , I'm sorry I didn't reply in time, but wanted to thank you for the fix. I confirm our sites are working fine again with the latest version.

Thank you!

@github-actions github-actions bot removed the needs:feedback This requires feedback to determine next steps. label Dec 26, 2023
@github-actions github-actions bot reopened this Dec 26, 2023
@github-project-automation github-project-automation bot moved this from Merged to In Progress in Open Source Practice Dec 26, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Merged in Open Source Practice Dec 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn’t working.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants