Skip to content

Updates max_execution_time#4274

Open
jonathanbossenger wants to merge 3 commits into
trunkfrom
fix-execution-timeout
Open

Updates max_execution_time#4274
jonathanbossenger wants to merge 3 commits into
trunkfrom
fix-execution-timeout

Conversation

@jonathanbossenger

Copy link
Copy Markdown

Related issues

How AI was used in this PR

Claude Code used to explore and propose the fix. Manually reviewed and tested by me

Proposed Changes

  • Sets the PHP max_execution_time to 0 (unlimited) instead of 30 seconds, to prevent long-running tasks from timing out when using PHP mode.

Testing Instructions

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 85e0c08 vs trunk

app-size

Metric trunk 85e0c08 Diff Change
App Size (Mac) 1355.70 MB 1355.70 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk 85e0c08 Diff Change
load 1040 ms 1105 ms +65 ms 🔴 6.3%

site-startup

Metric trunk 85e0c08 Diff Change
siteCreation 6989 ms 6989 ms 0 ms ⚪ 0.0%
siteStartup 2352 ms 2350 ms 2 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Native PHP sites default to a 30s max_execution_time (should match sandbox runtime)

2 participants