Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MEP TRELLO-2929 restore endpoint /api/ext/reports/siret/:siret #1902

Merged
merged 1 commit into from
Feb 26, 2025
Merged

MEP TRELLO-2929 restore endpoint /api/ext/reports/siret/:siret #1902

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion app/controllers/ReportToExternalController.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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),
Expand Down
18 changes: 14 additions & 4 deletions conf/routes
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 ########
##############################################################
Expand Down