php / php/php-src

Improve php-fpm -t

Abierto
#21,300 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Feature SAPI: fpm Status: Needs Triage
Lenguaje dominante
C
Estrellas
40.4k
Forks
8.2k
Merge medio
2 d 13 h
PR fusionados (30 d)
96

Descripción

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

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

No se nombran archivos fuente ni pruebas. Empieza reproduciendo el comportamiento reportado de pool.conf y php.ini con php-fpm -t; después, sigue el punto de entrada de la validación y el informe de errores existente. La tarea debería completarse con comprobaciones acordadas para configuraciones no válidas o inseguras, diagnósticos accionables y pruebas que las cubran.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
c, php
Área
backend
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.