php / php/php-src

Improve php-fpm -t

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

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

Feature SAPI: fpm Status: Needs Triage
主要言語
C
スター
40.4k
フォーク
8.2k
平均マージ
2日 13時間
マージ済み PR(30日)
96

説明

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

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

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

はじめの一歩

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

調査の方向性

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

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

評価

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

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

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