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.

Sign up for GitHub

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

Jump to bottom

Allow all 5xx HTTP Errors to Resume Upload Post-Processing #18081

Closed
getsource opened this issue Oct 23, 2019 · 0 comments · Fixed by #18106
Closed

Allow all 5xx HTTP Errors to Resume Upload Post-Processing #18081

getsource opened this issue Oct 23, 2019 · 0 comments · Fixed by #18106
Assignees
@youknowriad
Labels
[Feature] Media Anything that impacts the experience of managing media [Priority] High Used to indicate top priority items that need quick attention [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@getsource
Copy link
Member

Following #17474, during a conversation with the hosting community team, it was noted that some platforms use HTTP error codes other than 500 or 502, which were whitelisted.

cPanel, for example, uses an HTTP error of 508 to describe running out of resources.

For this reason, core/the media library was updated to allow errors 500-599 (inclusive) to retry.

It'd be great to support this with the editor in 5.3 as well, if possible, allowing more users to take advantage of resuming/retrying uploads.
@youknowriad youknowriad added [Feature] Media Anything that impacts the experience of managing media [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended labels Oct 23, 2019
@youknowriad youknowriad mentioned this issue Oct 23, 2019
Closed
@youknowriad youknowriad mentioned this issue Oct 25, 2019
Merged
@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Oct 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@youknowriad youknowriad

Labels
[Feature] Media Anything that impacts the experience of managing media [Priority] High Used to indicate top priority items that need quick attention [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow media upload post processing for all 5xx responses WordPress/gutenberg
2 participants
@youknowriad @getsource