ElementsProject / ElementsProject/elements

PSET is missing `asset` field in pegin witness data

Offen
#1,090 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C++
Sterne
1.2k
Forks
416
Ø Merge
1 T. 23 Std.
Gemergte PRs (30 T.)
15

Beschreibung

In PSET we have broken out all the fields of the `pegin_witness` field of a transaction input, *except* for the pegin asset type. I seem to recall the reason for this being that it was too complicated for Core to handle different pegin assets, and that no real blockchain would ever have multiple of these, so it had limited value.

However, for non-Elements finalizers, this field needs to exist, because it's used to construct the pegin witness.

It's fine if Elements rejects PSETs whose pegin assets don't match the consensus value, the field should just exist.

cc @achow101

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Lokalisieren Sie die PSET-Verarbeitung für pegin_witness-Daten von Transaktionseingaben und verfolgen Sie, wie Nicht-Elements-Finalizer diesen Witness verwenden. Fügen Sie eine Repräsentation des Pegin-Asset-Felds hinzu, wobei die Ablehnung von Assets erhalten bleibt, die nicht mit dem Konsenswert übereinstimmen, und überprüfen Sie anschließend die PSET-Serialisierung sowie das Verhalten bei der Finalizer-Konstruktion.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
blockchain
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.