FEAT: Filtrage caméras — adapter GetCamerasBySite aux droits utilisateur #135

New issue

Closed

opened 2026-08-18 08:57:24 +02:00 by nonobis · 0 comments

nonobis commented

2026-08-18 08:57:24 +02:00

Owner

Contexte

Le endpoint GET /api/cameras/by-site/{siteId} retourne toutes les caméras sans filtrage. Il faut appliquer le filtrage via ICameraAccessService.

Référence spec

Section 4.2 Permissions — filtrage des données.

État actuel

GetCamerasBySiteQueryHandler : retourne toutes les caméras du site
L'authorization est au niveau rôle, pas au niveau caméra individuelle

À faire

Injecter ICameraAccessService dans GetCamerasBySiteQueryHandler
Filtrer les résultats selon GetAccessibleCameraIdsAsync(userId, siteId)
Admin → pas de filtrage
Vérifier aussi GetCameraById (vérifier accès avant retour)
Vérifier StreamingHub.RequestStream (vérifier accès caméra, pas juste site)

Critères d'acceptation

GetCamerasBySite filtre selon les droits
Admin voit toutes les caméras
User avec accès site-wide voit toutes les caméras
User avec caméras spécifiques ne voit que celles-ci
GetCameraById vérifie l'accès
Tests passent

Fichiers concernés

src/Heimdall.Application/Cameras/Queries/GetCamerasBySite/GetCamerasBySiteQuery.cs
src/Heimdall.Application/Cameras/Queries/GetCamera/GetCameraQuery.cs
tests/Heimdall.Tests/Cameras/

## Contexte Le endpoint `GET /api/cameras/by-site/{siteId}` retourne toutes les caméras sans filtrage. Il faut appliquer le filtrage via `ICameraAccessService`. ## Référence spec Section 4.2 Permissions — filtrage des données. ## État actuel - `GetCamerasBySiteQueryHandler` : retourne toutes les caméras du site - L'authorization est au niveau rôle, pas au niveau caméra individuelle ## À faire 1. Injecter `ICameraAccessService` dans `GetCamerasBySiteQueryHandler` 2. Filtrer les résultats selon `GetAccessibleCameraIdsAsync(userId, siteId)` 3. Admin → pas de filtrage 4. Vérifier aussi `GetCameraById` (vérifier accès avant retour) 5. Vérifier `StreamingHub.RequestStream` (vérifier accès caméra, pas juste site) ## Critères d'acceptation - [ ] GetCamerasBySite filtre selon les droits - [ ] Admin voit toutes les caméras - [ ] User avec accès site-wide voit toutes les caméras - [ ] User avec caméras spécifiques ne voit que celles-ci - [ ] GetCameraById vérifie l'accès - [ ] Tests passent ## Fichiers concernés - `src/Heimdall.Application/Cameras/Queries/GetCamerasBySite/GetCamerasBySiteQuery.cs` - `src/Heimdall.Application/Cameras/Queries/GetCamera/GetCameraQuery.cs` - `tests/Heimdall.Tests/Cameras/`

nonobis added the

feature

priority-high

spec

labels

2026-08-18 08:57:29 +02:00

nonobis referenced this issue from a commit

2026-08-28 12:25:38 +02:00

feat(c cameras): enforce camera-level access filtering in GetCamerasBySite and GetCamera

nonobis referenced this issue from a pull request that will close it,

2026-08-28 12:25:55 +02:00

feat(c cameras): enforce camera-level access filtering in GetCamerasBySite and GetCamera #179