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.

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

Accès aux API REST métier pour les données d'avis pour tous les rôles des démarches #1208

Closed
lucaa opened this issue May 4, 2023 · 3 comments
Assignees
Labels
API mode connecté Amélioration ou problème qui impactent les fonctionnalités pour les utilisateurs connectés. patch-prod Correctif installé en production en dehors du cycle normal de deployement.
Milestone

Comments

@lucaa
Copy link
Collaborator

lucaa commented May 4, 2023

L'API métier ajoutée dans #1200 ne donne accès qu'aux administrateurs aux réponses des avis des démarches.

Ce ticket est pour implémenter la vérification des droits complète pour donner accès aux statistiques de réponses (impl. à base du schéma de droits de la plateforme).
L'objectif est que tout utilisateur qui peut accéder aux avis à partir de l'onglet des avis d'une démarche peut aussi y accéder par cette API.

@lucaa lucaa added mode connecté Amélioration ou problème qui impactent les fonctionnalités pour les utilisateurs connectés. API labels May 4, 2023
@lucaa lucaa added this to the 5.2 milestone May 4, 2023
@lucaa lucaa self-assigned this May 4, 2023
@lucaa lucaa modified the milestones: 5.2, 5.1.1 May 4, 2023
@lucaa
Copy link
Collaborator Author

lucaa commented May 26, 2023

L'objectif est que tout utilisateur qui peut accéder aux avis à partir de l'onglet des avis d'une démarche peut aussi y accéder par cette API.

En fait, ceci n'est pas tout à fait la logique correcte de cet accès: comme le service implémenté pour #1200 fourni l'accès aux données agrégées de la démarche, il n'y a pas de raison de protéger l'accès à ce service par la même règle que l'onglet des avis qui, lui, fourni l'accès aux données détaillées des avis (notamment les textes libres "verbatim" et les associations des avis).
Les données agrégées des avis sont celles affichées par les liens "graphes" des observatoires. Les liens "graphes" fournissent les données pour la période de l'observatoire en question mais à priori il n'y a pas de restriction dans la logique de droits de la plateforme pour la période pour laquelle ces graphes doivent être rendues publiques. J'ai vérifié l'implémentation en prod, on peut modifier les dates dans l'URL de graphes d'une démarche dans l'observatoire et générer la vue pour toute période souhaitée.

Ainsi, la question de rôles se pose plutôt par rapport à une problématique de charge de la plateforme par le service REST, il n'y a aucune contrainte d'accès à la donnée fournie par ce service.

lucaa added a commit that referenced this issue May 26, 2023
@lucaa
Copy link
Collaborator Author

lucaa commented May 26, 2023

Conformement au message ci-dessus, l'accés aux données agrégées est désormais implémenté pour tous les utilisateurs connectés.

Le choix de limiter l'accès aux utilisateurs connectés uniquement a été fait pour une raison de limitation d'accès aux services REST aux utilisateurs ayant un compte, pas pour des raisons de droits d'accès.

@lucaa lucaa closed this as completed May 26, 2023
@lucaa lucaa added the patch-prod Correctif installé en production en dehors du cycle normal de deployement. label May 26, 2023
@lucaa
Copy link
Collaborator Author

lucaa commented May 26, 2023

Cette modification a été installée en production et sera active demain, après le redemarrage programé de cette nuit.

@github-project-automation github-project-automation bot moved this to Done: Closed & Delivered in Développement Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API mode connecté Amélioration ou problème qui impactent les fonctionnalités pour les utilisateurs connectés. patch-prod Correctif installé en production en dehors du cycle normal de deployement.
Projects
Status: Done: Closed & Delivered
Development

No branches or pull requests

1 participant