spip/spip | 2 commits
Par Matthieu Marcillaud, le 26 septembre 2024 à 14h03min :
docs(changelog): #3919
Modifié
CHANGELOG.md
Détails : docs(changelog): #3919 (5ee36f96) · Validations · spip / spip · GitLab
==============================
Par Franck Rousseau, le 26 septembre 2024 à 14h02min :
fix: répare l’ajout de la config LDAP à l’install
Lors de l’installation, l’étape ldap5 fait deux suppositions fausses :
- le type d’authent est le dernier paramètre de spip_connect_db, « ’’); »
ligne 35 ; - le charset, qui est le dernier paramètre, est vide, ce qui n’est pas
toujours vrai.
Dans les deux cas la configuration est mauvaise, soit le type ldap est
placé au mauvais endroit, soit il n’est pas du tout ajouté.
Le problème de l’ajout du type de connexion à cette étape vient du fait
que la génération de cette config est faite ailleurs et qu’il n’est pas
raisonnable de pré-supposer de sa forme. Pour corriger ce problème on
ré-analyse la config temporaire, pour ensuite la regénérer avec les bons
paramètres, puis la substituer dans le fichier. Tout ceci est nécessaire
car on ne maitrise pas une partie des éléments ajoutés à ce fichier, on
ne peut donc pas le regénérer complètement.
Il reste un potentiel problème : la regex dans
analyse_fichier_connection tolère des espaces (\s) dans la ligne de
connexion, qui s’ils sont présents empêcheront le str_replace de
fonctionner correctement, puisqu’il ne seront pas présents dans la ligne
à substituer qui est regénérée à partir des paramètres individuels.
Fix: #3919
(cherry picked from commit 6e059210ea3abbb5ad7fe764fe3a387248c4e033)
Modifié
ecrire/install/etape_ldap5.php
Détails : fix: répare l'ajout de la config LDAP à l'install (657eaf32) · Validations · spip / spip · GitLab