Projet

Général

Profil

Anomalie #23843

[Maarch Parapheur Pro] Déduire la "core url" d'une variable d'environnement plutôt que des paramètres serveurs

Ajouté par Laurent CAMPION il y a environ un an. Mis à jour il y a environ un an.

Statut:
Résolu
Priorité:
3-Mineur
Assigné à:
Capucine JOURDAN
Version cible:
Début:
14/02/2023
Echéance:
Tags Parapheur:

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 :

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,

Historique

#2 Mis à jour par Emmanuel DILLARD il y a environ un an

Revu dans la mécanique 2301 :
paramètre url stocké en base , paramétrable via l'administration

#3 Mis à jour par Emmanuel DILLARD il y a environ un an

  • Statut changé de A qualifier à Complément d'Informations
  • Assigné à changé de Emmanuel DILLARD à Laurent CAMPION

#5 Mis à jour par Emmanuel DILLARD il y a environ un an

  • Assigné à changé de Laurent CAMPION à Capucine JOURDAN

#6 Mis à jour par Emmanuel DILLARD il y a environ un an

  • Priorité changé de 2-Sérieux à 3-Mineur

#7 Mis à jour par Emmanuel DILLARD il y a environ un an

  • Statut changé de Complément d'Informations à Résolu

Formats disponibles : Atom PDF