escrow::contribute and milestones::contribute accept new funds after deadline + GRACE_PERIOD, when anyone can already refund or cancel
I maintainer di solito rispondono entro 1 giorno
@urielit1313-creator ci sta già lavorando.
Dal 13/10/2026.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 25/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- rust
- Ambito
- blockchain
Direzione di ricerca
Read the contribute functions in the escrow and milestones contracts, then locate their existing tests. Check how the deadline and GRACE_PERIOD are represented and how InvalidDeadline is used. Done means both contracts reject contributions at or after the deadline and tests cover that behavior.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
Neither contribute checks the deadline. Once now >= deadline + GRACE_PERIOD, escrow's refund and milestones' cancel_milestone_after_deadline are permissionless, yet both contracts still take new contributions. A late sponsor's tokens go into an escrow or milestone that's already in its wind-down window: they can be refunded (or pro-rata redistributed, for milestones) by any third party in the next transaction, and the sponsor pays fees for a contribution that can never fund work.
Suggested fix
Reject contribute with InvalidDeadline (or a new DeadlinePassed) once now >= deadline, and test both contracts.
Related: #49, #56
- Lingua principale
- Rust
- Stelle
- 1
- Fork
- 102
- Merge medio
- 4h 12m
- PR unite (30g)
- 30
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
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 MergeFi/contracts
-
allocations map retains released issues — phantom ledger driftForse di nuovo libera @PINYOPATTANAWASANPORN l’ha presa 12 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 25/100
MergeFi/contracts#608 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Makefile deploy targets deploy the unoptimised .wasm — `stellar contract optimize` writes *.optimized.wasmForse già presa @urielit1313-creator l’ha presa 29 giorni fa. Apertabug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 25/100
MergeFi/contracts#567 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
keep_alive never refreshes instance storage, so a contract kept alive only via keep_alive can still archive Admin/Oracle/TreasuryForse già presa @urielit1313-creator l’ha presa 29 giorni fa. Apertabug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
MergeFi/contracts#566 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
[Regression on closed #254] milestones::deallocate still has no test asserting admin authorizationForse già presa @urielit1313-creator l’ha presa 29 giorni fa. Apertasecurity testing
Difficoltà 2/5 1-3 ore Idoneità per principianti 25/100
MergeFi/contracts#565 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
[Regression on closed #316] maintenance-pool::reclaim_deposit still has no test asserting the sponsor's own authorizationForse già presa @urielit1313-creator l’ha presa 29 giorni fa. Apertasecurity testing
Difficoltà 2/5 1-3 ore Idoneità per principianti 35/100
MergeFi/contracts#564 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di MergeFi/contracts
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
design-data-spec
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
adobe/spectrum-design-data#1744 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
USERID reads /etc/passwd directly instead of looking the user upForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertadefect
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno