le-phare / le-phare.github.io

The Faros site
https://faros.lephare.com
4 stars 2 forks source link

draft: docs: note about pcntl PHP extension #182

Open pierreboissinot opened 1 week ago

pierreboissinot commented 1 week ago

TLDR; j'ai ajouté une note indiquant que l'extension est activée par défaut dans les packages Debian.

Dans le cadre de la configuration d'une nouvelle machine suivant les prérequis 2024.07, je me suis demandé pour quelle raison figurait l'extension PHP pcntl.

Cette extension n'est pas installable mais est à activer lors du build de php.

Debian package php avec pcntl activé. Cf. code source Cette PR devrait résoudre cette issue https://github.com/le-phare/le-phare.github.io/issues/125

Cette extension est a priori utile pour symfony/messenger, je me suis demandé pourquoi cela ne figurait pas dans les dépendances symfony/messenger => parce que l'extension n'est pas disponible sur Windows et donc ce cas est géré. Voir détails.

@thislg étant donné cette issue, dois-je update que la dernière version dans ce cas ?

@lucasmirloup je n'ai pas pu tester en local que je n'ai rien cassé sur le site car j'ai une erreur SSL Screenshot from 2024-09-06 10-35-39 Saurais-tu m'aider stp ?

pierreboissinot commented 1 week ago

@thislg @lucasmirloup

Je vous propose d'ajouter aussi ces notes (c'est un extrait de playbook ne faites pas attention au format). Qu'en dites-vous ? Je fais ça dans une autre PR ou ici ?

#- 'php{{ php_version }}-ctype' # enabled by default https://www.php.net/book.ctype
# - 'php{{ php_version }}-iconv' # enabled by default https://www.php.net/manual/en/iconv.installation.php
#- 'php{{ php_version }}-pcre' # enabled by default https://www.php.net/book.pcre
#- 'php{{ php_version }}-session' # enabled by default https://www.php.net/book.session
#- 'php{{ php_version }}-simplexml' # enabled by default https://www.php.net/book.simplexml
#- 'php{{ php_version }}-tokenizer' # enabled by default https://www.php.net/book.tokenizer
thislg commented 1 week ago

Ça date mais je pense que https://github.com/le-phare/le-phare.github.io/issues/125 a été créée parce qu'on a eu besoin de demander explicitement l'activation de l'extension en CLI pour Messenger (sur un projet non hébergé sur Debian j'imagine).

Pour ce qui est de l'ajout du commentaire "activé par défaut sur Debian", je ne suis pas convaincu. On voudrait que l'hébergeur s'assure que tout soit installé / activé, Debian ou non. Et si Debian retire une extension comme ça peut arriver sur des nouvelles versions de php, le commentaire ne sera plus valide et induira en erreur.

lucasmirloup commented 1 week ago

Je n'ai pas encore eu le temps de tout lire, mais juste pour qu'on ait une trace : Le problème que j'ai rencontré à plusieurs reprises sur des projets est que l'hébergeur a bien activé l'extension, mais a désactivé toutes les fonctions PHP avec l'option PHP disable-functions, probablement pour des raisons de sécurité pré-paramétrées dans leurs outils d'hébergement.