maximus/simpl-liste
1
0
Fork 0
Code Issues 3 Pull requests Projects Releases 15 Packages Wiki Activity Actions

Intégrer @logto/rn avec expo-secure-store sur mobile #51

New issue
Closed
opened 2026-04-06 17:02:01 +00:00 by maximus · 1 comment
maximus commented 2026-04-06 17:02:01 +00:00
Owner
Copy link

Contexte

Le sync mobile (#40) utilise un placeholder pour l'auth (userId direct). Il faut intégrer le vrai flow OAuth2 Logto.

Taches

  • Installer @logto/rn et expo-secure-store
  • Créer un adaptateur de stockage custom pour @logto/rn basé sur expo-secure-store (ne pas utiliser AsyncStorage pour les tokens)
  • Configurer l'app Logto pour React Native (redirect URI avec scheme simplliste://)
  • Implémenter le flow de login dans Settings > Compte (bouton "Se connecter")
  • Stocker l'access token (15 min) et refresh token dans expo-secure-store
  • Implémenter la rotation automatique du refresh token
  • Intercepteur HTTP pour refresh transparent (capturer 401, rafraîchir, relancer)
  • Remplacer le placeholder dans syncClient.ts par le vrai JWT Bearer
  • Implémenter la réconciliation Inbox au premier merge (remap ID hardcodé → ID serveur)
  • Écran de première connexion : choix merge local ou reset serveur
  • Tester sur un device Android réel

Notes

  • Nécessite un device réel (le flow OAuth ne fonctionne pas dans l'émulateur sans configuration spécifique)
  • L'app Logto doit être configurée avec le scheme simplliste://callback

Dépendances

Depends on #48 (config Logto)

## Contexte Le sync mobile (#40) utilise un placeholder pour l'auth (userId direct). Il faut intégrer le vrai flow OAuth2 Logto. ## Taches - [ ] Installer `@logto/rn` et `expo-secure-store` - [ ] Créer un adaptateur de stockage custom pour `@logto/rn` basé sur `expo-secure-store` (ne pas utiliser AsyncStorage pour les tokens) - [ ] Configurer l'app Logto pour React Native (redirect URI avec scheme `simplliste://`) - [ ] Implémenter le flow de login dans Settings > Compte (bouton "Se connecter") - [ ] Stocker l'access token (15 min) et refresh token dans expo-secure-store - [ ] Implémenter la rotation automatique du refresh token - [ ] Intercepteur HTTP pour refresh transparent (capturer 401, rafraîchir, relancer) - [ ] Remplacer le placeholder dans `syncClient.ts` par le vrai JWT Bearer - [ ] Implémenter la réconciliation Inbox au premier merge (remap ID hardcodé → ID serveur) - [ ] Écran de première connexion : choix merge local ou reset serveur - [ ] Tester sur un device Android réel ## Notes - Nécessite un device réel (le flow OAuth ne fonctionne pas dans l'émulateur sans configuration spécifique) - L'app Logto doit être configurée avec le scheme `simplliste://callback` ## Dépendances Depends on #48 (config Logto)
maximus added this to the spec-simpl-liste-web milestone 2026-04-06 17:02:01 +00:00
maximus added the
status:ready
type:feature
source:human
 labels 2026-04-06 17:02:01 +00:00
maximus commented 2026-04-08 19:33:55 +00:00
Author
Owner
Copy link

L'intégration Logto est déjà complète dans le code actuel :

  • ✅ @logto/rn v1.1.0 + expo-secure-store installés
  • ✅ logtoConfig.ts avec endpoint, appId sl-mobile-native, redirect URI simplliste://callback
  • ✅ authToken.ts — bridge token getter entre React tree et syncClient
  • ✅ Settings: sign-in flow (logtoSignIn), sign-out, sync now
  • ✅ _layout.tsx: LogtoProvider, token getter registration, sync polling
  • ✅ Écran première connexion: choix merge local / reset serveur
  • ✅ initialMerge() + initialReset() dans syncClient.ts

Implémenté dans les commits du milestone. Fermeture comme déjà résolu.

L'intégration Logto est déjà complète dans le code actuel : - ✅ `@logto/rn` v1.1.0 + `expo-secure-store` installés - ✅ `logtoConfig.ts` avec endpoint, appId `sl-mobile-native`, redirect URI `simplliste://callback` - ✅ `authToken.ts` — bridge token getter entre React tree et syncClient - ✅ Settings: sign-in flow (`logtoSignIn`), sign-out, sync now - ✅ `_layout.tsx`: `LogtoProvider`, token getter registration, sync polling - ✅ Écran première connexion: choix merge local / reset serveur - ✅ `initialMerge()` + `initialReset()` dans syncClient.ts Implémenté dans les commits du milestone. Fermeture comme déjà résolu.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
issue-61-refresh-button-toolbar
issue-65-widget-sync-refresh
issue-60-fix-duplicate-inbox
fix/simpl-liste-32-widget-subtask-toggle
issue-29-widget-expand-perf
issue-25-edit-delete-subtasks
fix/simpl-liste-26-vulnerable-deps
fix/simpl-liste-23-widget-task-count
fix/simpl-liste-21-save-button-navigation
fix/simpl-liste-19-widget-add-button
fix/simpl-liste-11-widget-scroll
fix/simpl-liste-12-subtasks-completed-last
v1.6.1
v1.6.0
v1.5.2
v1.5.1
v1.5.0
v1.4.0
v1.3.0
v1.2.5
v1.2.4
v1.2.3
v1.2.2
v1.2.1
v1.2.0
v1.1.0
v1.0.1
v1.0.0
Labels
Clear labels   
source:analyste

Créée par le Super-Analyste

  
source:defenseur

Créée par l Escouade Défenseur

  
source:human

Créée par un humain

  
source:medic

Créée par le Medic (auto-détection)

  
status:approved

PR approuvée, prête à merger

  
status:blocked

Bloquée (question, dépendance, clarification)

  
status:in-progress

Un agent (Medic/Dev) travaille dessus

  
status:needs-fix

Reviewer demande des corrections

  
status:ready

Analysée et documentée, prête à prendre

  
status:review

PR créée, en attente du Reviewer

  
status:triage

Nouvelle issue, pas encore analysée

  
type:bug

Correction de bug → Medic

  
type:feature

Nouvelle fonctionnalité → Dev

  
type:infra

Config, CI/CD, déploiement → Dev

  
type:refactor

Refactoring / amélioration technique → Dev

  
type:schema

Modèle de données / migrations → Dev

  
type:security

Issue de sécurité → Medic ou Dev

No labels
source:analyste
source:defenseur
source:human
source:medic
status:approved
status:blocked
status:in-progress
status:needs-fix
status:ready
status:review
status:triage
type:bug
type:feature
type:infra
type:refactor
type:schema
type:security
Milestone
Clear milestone
No items
No milestone
spec-simpl-liste-web
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: maximus/simpl-liste#51
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?