From faf5d016de4b6024ff6e32729232ca4537b2be17 Mon Sep 17 00:00:00 2001
From: eletallbetagouv <107104509+eletallbetagouv@users.noreply.github.com>
Date: Wed, 26 Feb 2025 09:26:13 +0200
Subject: [PATCH] TRELLO-2929 restore endpoint /api/ext/reports/siret/:siret

---
 .../ReportToExternalController.scala           |  2 +-
 conf/routes                                    | 18 ++++++++++++++----
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/app/controllers/ReportToExternalController.scala b/app/controllers/ReportToExternalController.scala
index 628c5004..83be5f3a 100644
--- a/app/controllers/ReportToExternalController.scala
+++ b/app/controllers/ReportToExternalController.scala
@@ -116,7 +116,7 @@ class ReportToExternalController(
     )
   }
 
-  def searchReportsToExternalBySiret(siret: String) = Act.disabled.async { implicit request =>
+  def searchReportsToExternalBySiret(siret: String) = Act.securedbyApiKey.async { implicit request =>
     val qs = new QueryStringMapper(request.queryString)
     val filter = ReportFilter(
       siretSirenList = List(siret),
diff --git a/conf/routes b/conf/routes
index 8a7b2dfc..ca64658b 100644
--- a/conf/routes
+++ b/conf/routes
@@ -348,12 +348,22 @@ GET /api/ext/reports/:uuid        controllers.ReportToExternalController.getRepo
 # On leur crée cet endpoint pour ne plus qu'il utilise celui legacy qui n'était pas authentifié
 GET /api/ext/reports/files/used/:fileId/:filename     controllers.ReportToExternalController.downloadFileUsedInReport(fileId: ReportFileId, filename)
 
-# Ces deux endpoints semblaient inutilisés
-# Nous les avons maintenant (2025-02) désactivés dans le code (ils renvoient un code d'erreur)
-# A terme, si personne ne se plaint on pourra les supprimer entièrement
-GET /api/ext/v2/reports           controllers.ReportToExternalController.searchReportsToExternalV2()
+# Cet endpoint apparait inutilisé dans les logs
+# Pourtant dans une discussion avec Georges Lagresa et Laurent Fontaine
+# ils disent qu'il est utilisé dans "SORA" :
+#     "C’est une fonctionnalité qui est offerte aux agents qui leur
+#     permet d’obtenir une liste de signalements selon le SIRET de l’ETS en cours d’analyse."
+# Comme il n'y a rien dans les logs des 30 derniers jours, c'est probablement une fonctionnalité
+# pratiquement inutilisée. On garde l'endpoint quand même.
+# Note : "SORA" utilise la même api key que ReponseConso
 GET /api/ext/reports/siret/:siret controllers.ReportToExternalController.searchReportsToExternalBySiret(siret: String)
 
+# Cet endpoint apparait inutilisé dans les logs
+# Visiblement il était censé remplacer une version précédente mais ReponseConso n'a jamais fait la migration
+# On l'a maintenant (2025-02) désactivé dans le code (il renvoit un code d'erreur)
+# A terme, si personne ne se plaint on pourra le supprimer entièrement
+GET /api/ext/v2/reports           controllers.ReportToExternalController.searchReportsToExternalV2()
+
 ##############################################################
 ######## AUTRES ENDPOINTS SANS FRONTEND CORRESPONDANT ########
 ##############################################################