Improve php-fpm -t

オープン
#21,300 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
5/5
見積もり時間
1週間以上
初心者へのやさしさ
25/100
issue の種類
機能追加
明瞭さ
説明が足りない
活発さ
停滞
技術スタック
c, php
領域
backend

調査の方向性

ソースファイルやテストは指定されていません。まず php-fpm -t で報告された pool.conf と php.ini の挙動を再現し、次にバリデーションのエントリーポイントと既存のエラー報告を追跡してください。完了条件には、無効または安全でない設定に対する合意済みのチェック、それに対する実行可能な診断、およびそれらをカバーするテストを含める必要があります。

索引モデルが issue の本文から書いたものです。

説明

Feature SAPI: fpm Status: Needs Triage
Description

if pool.conf contains:

php_admin_value[memory_limit] = 128MB

it does not overwrite the global php.ini,except memory_limit is not set in php.ini, than 128M will be used.

this is either coaccidently the php build in default
OR
an emergc fall back in php-fpm to as in "better try this than nothing"

Suggestion:

add checks to "php-fpm -t" to find:

a) wrong arguments
b) stupid configs which will cause php-fpm master to crash in an endless loop with "code 70"
c) "if you know something, say something"

Example:

if "process.max = 80" is lower than an start servers together ( or other sanity checks ) , than say something and don't lure the using user in a false hope of "it can work" , just to endless crash with "code 70" which tells you: i crash, but not why i crashed" which is VVVVVVEEEEEEERRRYYYY annoying btw.

PHP 8.4.18 (fpm-fcgi) (built: Feb 10 2026 17:48:03) (NTS gcc x86_64)
Copyright (c) The PHP Group
Built by Fedora Project
Zend Engine v4.4.18, Copyright (c) Zend Technologies
with Zend OPcache v8.4.18, Copyright (c), by Zend Technologies

主要言語
C
スター
40.4k
フォーク
8.2k
平均マージ
2日 15時間
マージ済み PR（30日）
103

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

php/php-src のほかの issue

php/php-src の issue をすべて見る

似ている issue

C の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。