Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

activate pcntl extension in CLI #125

Open
thislg opened this issue Nov 21, 2023 · 8 comments · May be fixed by #189
Open

activate pcntl extension in CLI #125

thislg opened this issue Nov 21, 2023 · 8 comments · May be fixed by #189
Assignees

Comments

@thislg
Copy link
Member

thislg commented Nov 21, 2023

Add pcntl extension activation requirement in CLI (for Symfony Messenger)

@pierreboissinot
Copy link
Member

@thislg il faudrait ajouter une sections "fonctions non désactivées", en commençant par celles de cette extension.

Je me demande si c'est optionnel ou non, il me semble que dès qu'on a du mail on a messenger donc a priori oui vu que dans un projet faros on a forcément du mail.

Qu'en pensez vous ?

@lucasmirloup
Copy link
Member

Ça fait plusieurs fois que le problème arrive donc je valide 👍 @pierreboissinot

@thislg
Copy link
Member Author

thislg commented Oct 15, 2024

Messenger n'est pas une dépendance nécessaire à Faros et n'est pas obligatoire pour envoyer des emails avec Symfony Mailer, en fait Messenger n'est utilisé que s'il est installé, sinon ça envoie simplement en synchrone. Par contre c'est ce qu'on a tendance à utiliser systématiquement pour faire du traitement asynchrone y compris l'envoi des emails donc c'est cohérent de demander les fonctions nécessaires dans les prérequis.

@pchabbert
Copy link
Contributor

Si l'extension n'apporte pas de problème a être activée même si elle n'est pas utilisée je ne vois pas de soucis non plus 👍

@pierreboissinot
Copy link
Member

merci pour vos retours , je démarre une PR

@pierreboissinot
Copy link
Member

Les retours d' @alexandrebouillet
image

@lucasmirloup
Copy link
Member

Hello @alexandrebouillet

On est d'accord que le disable_functions dans ton message est celui que tu as demandé à l'hébergeur de mettre ?
Le projet en question utilise-t-il Symfony Messenger ?

À première vue j'aurais tendance à penser que ces fonctions pcntl désactivées sont nécessaires pour Symfony Messenger, mais ce n'est peut-être pas le cas.

@pierreboissinot
Copy link
Member

Hello @alexandrebouillet

On est d'accord que le disable_functions dans ton message est celui que tu as demandé à l'hébergeur de mettre ? Le projet en question utilise-t-il Symfony Messenger ?

À première vue j'aurais tendance à penser que ces fonctions pcntl désactivées sont nécessaires pour Symfony Messenger, mais ce n'est peut-être pas le cas.

On est d'accord que le disable_functions dans ton message est celui que tu as demandé à l'hébergeur de mettre ?

Non, @alexandrebouillet me corrigera si j'ai mal compris mais il s'agit des func désactivées par l'hébergeur.

À première vue j'aurais tendance à penser que ces fonctions pcntl désactivées sont nécessaires pour Symfony Messenger, mais ce n'est peut-être pas le cas.

Oui, d'où un warning et non une erreur https://github.com/le-phare/le-phare.github.io/pull/189/files#diff-d469def2c8383934ca736810fa3ffc2c362617570ecf2184f2fc12d15f229fb9R31

PS: @lucasmirloup j'imagine que tu n'as pas lu la PR, je t'ai ajouté en reviewer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants