Anomalie #23843
fermé[Maarch Parapheur Pro] Déduire la "core url" d'une variable d'environnement plutôt que des paramètres serveurs
Description
Bonjour Emmanuel,
Je te transmet ci-après une demande client :
Lors de tests liés à l'envoi des notifications dans Maarch Parapheur Pro il apparaît des ips et ports internes dans certains mails :
![](Screenshot from 2023-02-14 10-27-25.png)
Ce comportement est de nature à perturber les utilisateurs et à empêcher le lien de fonctionner puisque ces ips n'ont pas de sens en amont des reverse-proxy de notre centre-serveur.
En regardant le code utilisé pour calculer l'url de base (core url) dans le fichier src/core/controllers/UrlController.php, je comprends que les informations sont déduites des variables serveurs fournies par Apache ce qui est hasardeux dans le cas (comme le nôtre) d'une architecture containérisée associée à plusieurs reverse-proxys en cascade.
J'ai, pour le moment, produit un fix consistant à court-circuiter complètement le calcul en question pour déduire l'url entière d'une variable d'environnement INSTANCE_URL fournie à l'application (dans notre cas via le docker-compose.yml).
Ce ticket a pour objet de suggérer que c'est une modification qui aurait tout intérêt à être pérénisée dans le code éditeur : en l'état mon "fix" est fragile et pourrait être rendu nul par des modifications à la source du fichier UrlController.php tandis que côté Maarch il me semble y avoir de moins en moins d'intérêt à maintenir du code complexe pour déduire une url de base sachant que les architectures clientes sont de plus en plus containérisées et facilement paramétrables.
Le contenu du fichier UrlController.php pourrait ainsi se résumer à :
namespace SrcCore\controllers;
class UrlController
{
public static function getCoreUrl()
{
return getenv("INSTANCE_URL");
}
}
Bien cordialement,
Fichiers
Mis à jour par Emmanuel DILLARD il y a presque 2 ans
Revu dans la mécanique 2301 :
paramètre url stocké en base , paramétrable via l'administration
Mis à jour par Emmanuel DILLARD il y a presque 2 ans
- Statut changé de A qualifier à Complément d'Informations
- Assigné à changé de Emmanuel DILLARD à Laurent CAMPION
Mis à jour par Emmanuel DILLARD il y a presque 2 ans
- Assigné à changé de Laurent CAMPION à Capucine JOURDAN
Mis à jour par Emmanuel DILLARD il y a plus d'un an
- Priorité changé de 2-Sérieux à 3-Mineur
Mis à jour par Emmanuel DILLARD il y a plus d'un an
- Statut changé de Complément d'Informations à Résolu