Projet

Général

Profil

Actions

Anomalie #23843

fermé

[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 presque 2 ans. Mis à jour il y a environ un mois.

Statut:
Clôturé
Priorité:
3-Mineur
Assigné à:
Capucine JOURDAN
Version cible:
Début:
14/02/2023
Echéance:
Projet 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 :

![](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 plus d'un an

  • 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

Mis à jour par Emmanuel DILLARD il y a environ un mois

  • Statut changé de Résolu à Clôturé
Actions

Formats disponibles : Atom PDF