Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Follow-up to #6676: Version-guarded delete: mixed-version owners, fallback-read faults, teardown classification, wiring test

Chiusa
  • Qualcosa non va?
#6,750 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Una pull request collegata è già stata integrata.

  • #6676 di @rbuergi — integrata

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
18/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
csharp
Ambito
backend, distributed-systems

Direzione di ricerca

Start from the version-guarded deletion path referenced in the issue: MeshExtensions.cs around lines 4209-4211 (production wiring) and the legacy teardown branch near lines 6413-6427, plus the RideOutRecyclingLeaf handling of ShuttingDown. Each finding needs a failing test first, so begin by writing a test that drives a real guarded DeleteNodeRequest and pauses after the owner verdict. Done means the four checklist items are fixed and that test shows an own write is refused until the delete completes.

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

Descrizione

bug feature:node-delete sev:M

Follow-up to #6676 (review findings deferred from that pull request; Refs #6676).

  • During a roll, an old owner skips the unknown ExpectedVersion member (UnmappedMemberHandling.Skip) and answers Ok(). Use a distinct request type, or require an echoed capability and map a missing echo to Unavailable.
  • A faulting fallback storage read escapes and is classified Unknown/ValidationFailed; map it to Unavailable.
  • The blanket catch turns owner teardown into Unavailable, so RideOutRecyclingLeaf never sees ShuttingDown; classify teardown as the legacy branch does (lines 6413-6427).
  • The new tests open BeginVersionGuardedDeletion directly; removing the production wiring (MeshExtensions.cs:4209-4211) leaves them green. Drive a real guarded DeleteNodeRequest, pause after the owner verdict and assert an own write is refused until the delete completes.

These are automated-review findings, assessed from the diff and not reproduced; each needs a failing test before its fix. Deferred from the PR because CI is saturated and none of them is a security hole, data loss or a broken build/test.

Lingua principale
C#
Stelle
12
Fork
5
Merge medio
6h 36m
PR unite (30g)
951

Preparare l'ambiente

  • Nessun Dockerfile né file Docker Compose
  • Ha un modello di pull request
  • Nessuna guida per i contributori

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 Systemorph/MeshWeaver

Tutte le issue di Systemorph/MeshWeaver

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.