Skip to content

Comments

Fixes#13

Merged
cappuc merged 3 commits intomasterfrom
dev
Feb 17, 2026
Merged

Fixes#13
cappuc merged 3 commits intomasterfrom
dev

Conversation

@cappuc
Copy link
Contributor

@cappuc cappuc commented Feb 17, 2026

  • removed already installed extensions
  • removed excimer (it cause segfault with frankenphp)
  • reduced JIT buffer size (can cause segfault when too big)

Copilot AI review requested due to automatic review settings February 17, 2026 11:02
@cappuc cappuc merged commit 3d8f678 into master Feb 17, 2026
22 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR optimizes the PHP Docker image by removing redundant extension installations and fixing a compatibility issue with FrankenPHP. The changes reduce image build time and prevent potential runtime segfaults.

Changes:

  • Removed built-in PHP extensions (iconv, pdo_sqlite, phar, posix, simplexml, sodium) from the installation list as they are already included in PHP core
  • Removed excimer extension due to segfault issues with FrankenPHP variant
  • Reduced JIT buffer size from 100M to 32M to prevent potential segfaults under high memory pressure
  • Fixed indentation in nginx configuration conditional block

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant