TRELLO-2929 disable unused api endpoints

app/controllers/BaseController.scala

@@ -77,9 +77,16 @@ abstract class ApiKeyBaseController(
     authenticator
   ) andThen new ErrorHandlerActionFunction[ConsumerRequest]()
 
+  private def disabledActionFilter = new ActionFilter[Request] {
+    override protected def executionContext = ec
+    override protected def filter[A](request: Request[A]): Future[Option[Result]] =
+      Future.successful(Some(Gone))
+  }
+
   trait ActDslApiKey {
     val securedbyApiKey = securedByApiKeyAction
-
+    // we disable some endpoints that seem completely unused.
+    val disabled = Action.andThen(disabledActionFilter)
   }
   val Act = new ActDslApiKey {}
 }

app/controllers/ReportToExternalController.scala

@@ -85,7 +85,7 @@ class ReportToExternalController(
     )
   }
 
-  def searchReportsToExternalV2() = Act.securedbyApiKey.async { implicit request =>
+  def searchReportsToExternalV2() = Act.disabled.async { implicit request =>
     val qs = new QueryStringMapper(request.queryString)
     val filter = ReportFilter(
       siretSirenList = qs.string("siret").map(List(_)).getOrElse(List()),
@@ -116,10 +116,7 @@ class ReportToExternalController(
     )
   }
 
-  /** @deprecated
-    *   Keep it for retro-compatibility purpose but searchReportsToExternal() is the good one.
-    */
-  def searchReportsToExternalBySiret(siret: String) = Act.securedbyApiKey.async { implicit request =>
+  def searchReportsToExternalBySiret(siret: String) = Act.disabled.async { implicit request =>
     val qs = new QueryStringMapper(request.queryString)
     val filter = ReportFilter(
       siretSirenList = List(siret),

conf/routes

@@ -333,16 +333,10 @@ GET           /api/mobileapp/requirements                             controller
 ########## FOR EXTERNAL SIs ############
 ########################################
 
-# celui-ci est marqué deprecated dans le code, qui dit d'utiliser /api/ext/reports (lui même remplacé par V2 ?)
-# pas utilisé dans les 30 derniers jours d'après New Relic
-GET /api/ext/reports/siret/:siret controllers.ReportToExternalController.searchReportsToExternalBySiret(siret: String)
 
 # très utilisé. Principalement pour filtrer sur le tag ReponseConso, donc ça doit être RéponseConso qui l'utilise ?
 GET /api/ext/reports              controllers.ReportToExternalController.searchReportsToExternal()
 
-# pas utilisé dans les 30 derniers jours d'après New Relic
-# curieux, alors que c'est censé être une V2 par rapport au précédent ?
-GET /api/ext/v2/reports           controllers.ReportToExternalController.searchReportsToExternalV2()
 
 # utilisé une fois par jour vers minuit
 GET /api/ext/reports/extract      controllers.DataEconomieController.reportDataEcomonie()
@@ -354,6 +348,12 @@ 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()
+GET /api/ext/reports/siret/:siret controllers.ReportToExternalController.searchReportsToExternalBySiret(siret: String)
+
 ##############################################################
 ######## AUTRES ENDPOINTS SANS FRONTEND CORRESPONDANT ########
 ##############################################################