betagouv / rdv-service-public

Prise de RDV pour les services publics
https://rdv.anct.gouv.fr
GNU Affero General Public License v3.0
14 stars 2 forks source link

Echec des envois de mails de statistiques #3701

Open victormours opened 1 year ago

victormours commented 1 year ago

ℹ️ Contexte

Actuellement, pour télécharger des statistiques de rdv, les référentes cliquent sur un lien sur la liste des rdv pour qu'un export soit envoyé. On leur affiche ce message de confirmation :

Screenshot 2023-07-28 at 14 42 48

Il arrive souvent que la pièce jointe du mail soit trop grosse, et que le mail n'arrive pas à destination (voir https://sentry.incubateur.net/organizations/betagouv/issues/55725/?environment=production&query=is%3Aunresolved&referrer=issue-stream). On a alors typiquement ce message d'erreur : 552 Mail size too large. Maximum mail size limit is 20MB

On a eu ce cas récemment avec les référentes suivantes : https://www.rdv-solidarites.fr/super_admins/agents/12622 et https://www.rdv-solidarites.fr/super_admins/agents/5496

(voir aussi les erreurs Sentry : https://sentry.incubateur.net/organizations/betagouv/issues/56075/?project=74&referrer=webhooks_plugin, qui concernent parfois le pipe unix qui transfère les données avant même que le mail soit envoyé, mais qui finit généralement par faire l'envoi)

💡 Solution envisagée

Le plus simple : ajouter dans le message de confirmation que si elles ne reçoivent pas le mail, elles peuvent réessayer en ajoutant des limites de dates pour exporter moins de rdv à la fois

Plus compliqué : uploader l'export sur un hébergeur de fichier sécurisé (attention, ces exports contiennent énormément de données personnelles), et leur envoyer un lien de téléchargement. C'est pas trivial à mettre en place, et ça pose beaucoup de questions de sécurité des données, donc c'est sans doute pas une très bonne idée.

La version qui nécessite plus de temps de discussion mais qui simplifie le produit : voir avec elles pourquoi elles font ces exports, et si on ne pourrait pas diminuer la quantité de données personnelles qu'ils contiennent, ce qui permettrait d'avoir des fichiers plus petits pour le même nombre de rdv. Allez voir un export depuis la démo : on sort beaucoup de données personnelles des agents et des usagers. Je suis pas sûr que ça soit une bonne chose.

Ma reco serait d'implémenter la première solution, et de faire la troisième dans le cadre de notre amélioration de la gestion des données personnelles.

Pour rappel, voici la liste des colonnes dans l'export :

   "année",
    "date prise rdv",
    "heure prise rdv",
    "origine",
    "date rdv",
    "heure rdv",
    "service",
    "motif",
    "contexte",
    "statut",
    "lieu",
    "professionnel.le(s)",
    "usager(s)",
    "commune du premier responsable",
    "au moins un usager mineur ?",
    "résultat des notifications",
    "Organisation",
    "date naissance",
    "code postal du premier responsable",
    "créé par",
    "email(s) professionnel.le(s)",
victormours commented 1 year ago

Le texte qu'on pourrait mettre :

Votre export est en cours de création : il sera envoyé à martine@demo.rdv-solidarites.fr dès qu'il sera prêt.

Il est possible que votre boîte mail ne permette pas qu'on lui envoie des fichiers trop lourds. Si vous ne recevez pas l'export dans les minutes qui viennent, essayez d'exporter moins de rendez-vous à la fois en les filtrant sur une période plus courte.

Pour plus d'aide, vous pouvez aussi nous contacter à l'adresse support@rdv-solidarites.fr.