[codex] Fix bekende personen feature review findings #50

Merged

janpeter merged 4 commits from codex/duplicate-cleanup-review-fixes into main

2026-06-13 17:11:45 +02:00

janpeter commented

2026-06-13 17:10:14 +02:00

Owner

Summary

Require session auth before serving or proxying Immich person asset media.
Make the Immich person browser/search query the full local person set instead of a capped in-memory list.
Stop exposing physical filesystem paths in Immich person media details.
Add the code review and implementation plan markdown docs.

Why

The known-people feature review found unauthenticated media access, incomplete person discovery for larger Immich libraries, and local path leakage in the client detail model.

Validation

npm test
npm run build

## Summary - Require session auth before serving or proxying Immich person asset media. - Make the Immich person browser/search query the full local person set instead of a capped in-memory list. - Stop exposing physical filesystem paths in Immich person media details. - Add the code review and implementation plan markdown docs. ## Why The known-people feature review found unauthenticated media access, incomplete person discovery for larger Immich libraries, and local path leakage in the client detail model. ## Validation - npm test - npm run build

janpeter added 4 commits

2026-06-13 17:10:14 +02:00