mimblewimble/grin

Fence off txhashset validation in its own process

Offen

#2.741 geöffnet am 10.04.2019

Auf GitHub ansehenContributor Guide
 (11 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (991 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (4.876 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

When downloaded from the network on sync, a txhashset archive should be handled with more care than we do now, as it could be forged to mess things up in all sorts of ways. We should fence this off in a different process until the validation code is completely happy with what we were provided. Something like chroot, that limits the hard drive visibility of the forked process would also be great but may be hard to achieve across platforms.

Contributor Guide

Research-Richtung
Untersuche den aktuellen txhashset Validierungscode in der grin Codebasis (wahrscheinlich in src/core oder src/sync). Untersuche Rusts std::process API zum Erzeugen von Kindprozessen. Recherchiere plattformübergreifende Möglichkeiten, den Dateisystemzugriff einzuschränken (z. B. mit Containern, seccomp oder plattformspezifischem chroot). Überprüfe die 11 Kommentare im Issue auf etwaige Entwurfsvorschläge oder Hindernisse.
Tech Stack
rust
Domain
backendsecurity
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40