fix: raise resource limits for synchronous pre-update backup (white screen) #236

Merged

jmiller merged 2 commits from fix/preupdate-wsod-limits into dev 2026-07-06 03:34:20 +00:00

Conversation 0 Commits 2 Files Changed 15

+24 -12

jmiller commented 2026-07-06 03:34:02 +00:00

Owner

Copy Link

Copy Source

Immediate safety net for the white screen reported when clicking Update with backup_before_update on.

Cause

runPreActionBackup() (onExtensionBeforeUpdate/onExtensionBeforeUninstall) runs a full backup synchronously inside the update request with default PHP limits. On a large site (suite.dev is ~123 MB) it exceeds max_execution_time/memory_limit mid-backup → white screen of death.

Fix

Raise set_time_limit(0) / max_execution_time / memory_limit=512M / ignore_user_abort(true) before the synchronous backup — mirroring the web-cron handler which already does this.

This is a targeted safety net. The larger fix (full-screen backup screen + server-side redirect for core Joomla updates, so no synchronous backup runs in that request at all — Akeeba-style) is coming in a follow-up; these limits remain the guard for the extension-update/uninstall path.

Verify

• php -l clean.

https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

Immediate safety net for the white screen reported when clicking Update with `backup_before_update` on. ## Cause `runPreActionBackup()` (`onExtensionBeforeUpdate`/`onExtensionBeforeUninstall`) runs a **full backup synchronously inside the update request** with default PHP limits. On a large site (suite.dev is ~123 MB) it exceeds `max_execution_time`/`memory_limit` mid-backup → white screen of death. ## Fix Raise `set_time_limit(0)` / `max_execution_time` / `memory_limit=512M` / `ignore_user_abort(true)` before the synchronous backup — mirroring the web-cron handler which already does this. This is a targeted safety net. The larger fix (full-screen backup screen + server-side redirect for **core Joomla updates**, so no synchronous backup runs in that request at all — Akeeba-style) is coming in a follow-up; these limits remain the guard for the extension-update/uninstall path. ## Verify - `php -l` clean. https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

jmiller added 1 commit 2026-07-06 03:34:03 +00:00

fix: raise resource limits for synchronous pre-update backup (WSOD) ... 49463eb747

The pre-update/uninstall backup runs synchronously inside the extension
update request. Without raising limits, a large site exceeds
max_execution_time/memory_limit mid-backup and the update page
white-screens. Raise set_time_limit(0)/memory_limit/ignore_user_abort
like the web-cron path.

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

jmiller added 1 commit 2026-07-06 03:34:13 +00:00

chore(version): pre-release bump to 02.58.12-dev [skip ci] 064963da9c

jmiller merged commit 094787d26d into dev 2026-07-06 03:34:20 +00:00

jmiller deleted branch fix/preupdate-wsod-limits 2026-07-06 03:34:21 +00:00

jmiller referenced this pull request 2026-07-06 03:56:27 +00:00

feat: full-screen backup screen for pre-update + Backup Now (Akeeba-style), fixes white screen #238

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

component: admin

Admin UI and views

component: api

REST API

component: engine

Backup/restore engine

component: remote

Remote storage (FTP/GDrive)

component: scheduler

Scheduled tasks

bug

Something is not working

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

question

More information is needed

wontfix

This won't be fixed

No labels

Priority -

Type -

Milestone

No items

No Milestone

Assignees

Clear assignees

jmiller (Jonathan Miller)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/MokoSuiteBackup#236