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

[PHP] Restore /internal files and Filesystem mounts after hotswapPhpRuntime is called #2119

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

adamziel
Copy link
Collaborator

@adamziel adamziel commented Jan 9, 2025

Without this PR, hotSwapPHPRuntime() does not:

  • Recreate the filesystem mounts
  • Restore the platform-level plugins stored in the /internal (e.g the SQLite integration plugin)

Which results in WordPress losing access to local files and the database connectivity everytime the PHP runtime is rotated.

This PR ensures the mounts are re-created and the platform-level plugins preserved.

Closes #1596

Testing instructions

This PR comes with tests, so check the CI logs.

 Re-create the mounts destroyed during the hotSwapPHPRuntime call.
 Closes #1596

  ## Context
  Without this PR, `hotSwapPHPRuntime()` does not:   * Recreate the filesystem mounts * Restore the platform-level plugins stored in the /internal (e.g the
   SQLite integration plugin)

 Which results in WordPress losing access to local files and the database connectivity.

 This PR ensures the mounts are re-created and the platform-level plugins
 preserved.

  ## Testing instructions

 This PR comes with tests, so check the CI logs.
@adamziel adamziel added [Type] Bug An existing feature does not function as intended [Package][@php-wasm] Web [Package][@php-wasm] Node labels Jan 9, 2025
@adamziel adamziel changed the title [PHP] Remount NODEFS and other mounts after hotswapPhpRuntime is called [PHP] Restore /internal files and Filesystem mounts after hotswapPhpRuntime is called Jan 9, 2025
@adamziel adamziel merged commit e4b3c87 into trunk Jan 10, 2025
10 checks passed
@adamziel adamziel deleted the fix-lost-db-connection branch January 10, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package][@php-wasm] Node [Package][@php-wasm] Web [Type] Bug An existing feature does not function as intended
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Intermittent SQLite Database Disconnects
1 participant