Disconnect and per-instance sign-out still post a logout through the Fabric Connect proxy, which can never succeed

Aperta Adatta ai principianti
  • Qualcosa non va?
#1,974 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
70/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
react, typescript
Ambito
frontend

Direzione di ricerca

Inizia da src/features/cluster/ClusterHome.tsx (onDisconnect) e src/features/cluster/signOutOfInstance.ts, i due punti che inviano ancora il logout tramite proxy, e leggi come connectsThroughProxy(id) e onInstanceLogoutSubmit determinano la modalità del client. È completato quando nessuno dei due percorsi invia il logout attraverso il proxy di Fabric Connect, Disconnect non attende più quella richiesta e lo stato locale viene comunque azzerato come prima. ClusterCard.onSignOutClick è fuori ambito.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

rum

Summary

HarperFast/studio#1973 stops the app sign-out sweep and self-hosted cluster removal from posting { operation: 'logout' } through the Fabric Connect proxy, where the host manager calls the instance with no cookie or token and Harper has no session of the user's to end (16 of 24 answered 500 in 30 days of RUM). Two user-initiated paths still post it:

  • ClusterHome.onDisconnect (src/features/cluster/ClusterHome.tsx) — the Disconnect button on a proxied cluster or instance. It awaits the proxied logout before clearing local state, through a client that keeps the 60 s timeout and the gateway-error retries.
  • signOutOfInstance (src/features/cluster/signOutOfInstance.ts) — the per-instance sign-out, same shape.

ClusterCard.onSignOutClick is not affected: on the clusters list there is no route id, so its client resolves to the cloud slot and is never proxied.

Fix

Either call connectsThroughProxy(id) at each site before building the client (both build theirs while the Fabric Connect flag is still set, so the predicate reads live), or move the guard into onInstanceLogoutSubmit by reading the mode the built client resolved to. Both paths already clear local state whether or not the POST succeeds, so skipping it changes nothing the user sees except that Disconnect returns at once.

Raised by the cross-model review of the #1973 fix as outside that diff.

Review-Deferral: 1c2b8342a1512922 severity=minor category=reliability

🤖 Generated by Claude Code (Claude Mythos 5.1); posted via @dawsontoth.

Lingua principale
TypeScript
Stelle
5
Fork
4
Merge medio
1g 19h
PR unite (30g)
144

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di HarperFast/studio

Tutte le issue di HarperFast/studio

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.