fix(breg-review): refuse sign-in limits that cannot complete one flow
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- rust
- Ambito
- authentication
Direzione di ricerca
Inizia in crates/registry-breg-review/src/config.rs, quindi segui gli addebiti del limite per /signin e /signin/callback. Verifica se la correzione impone un burst di almeno due richieste oppure addebita l’intero flusso una sola volta. Il lavoro è completato quando una configurazione con ricarica lenta non può lasciare un sign-in bloccato tra il suo avvio e il callback.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
A sign-in charges limits.globalSignIn twice (/signin and /signin/callback), but crates/registry-breg-review/src/config.rs accepts burst: 1 with any refill rate. With burst: 1 and a slow refill, the start spends the only token, the immediate callback is throttled (which now ends the pending flow), and no one can sign in. Require a burst of at least two, or charge the complete flow once.
Found by Codex review on #1534 at 5ac4feb56; deferred under the rule that only security or data-integrity findings block that merge. breg-mcp and breg-review are held out of releases (BREG_SERVICES_FIRST_RELEASE = None), so this should land before that constant names a first release.
- Lingua principale
- Rust
- Stelle
- 2
- Fork
- 0
- Merge medio
- 8h 10m
- PR unite (30g)
- 222
Preparare l'ambiente
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di registrystack/registry-stack
-
area:casework bug criticality:p3 triage:needs-implementation
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
registrystack/registry-stack#1729 ·
I maintainer di solito rispondono entro 1 giorno
-
area:breg bug criticality:p3 triage:needs-implementation
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
registrystack/registry-stack#1682 ·
I maintainer di solito rispondono entro 1 giorno
-
area:casework bug criticality:p3 triage:needs-implementation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
registrystack/registry-stack#1669 ·
I maintainer di solito rispondono entro 1 giorno
-
agent-ready area:platform criticality:p3 documentation triage:needs-implementation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
registrystack/registry-stack#1639 ·
I maintainer di solito rispondono entro 1 giorno
-
area:evidence bug criticality:p3 triage:needs-implementation
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
registrystack/registry-stack#1620 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di registrystack/registry-stack
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 3 giorni
-
state:triage-needed
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Automattic/harper#4503 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
I maintainer di solito rispondono entro 2 giorni