signmessage
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- cpp
- Ambito
- blockchain
Direzione di ricerca
Inizia con src/util/message.cpp e gli entry point signmessage/verifymessage, quindi confronta il loro comportamento con i casi Elements/Liquid e il riepilogo del wallet nell’issue. Il lavoro è completato quando si sarà raggiunta una decisione del progetto sul prefisso del messaggio, sulla gestione delle blinding keys, sul supporto a non-p2pkh e sulla permanenza della firma dei messaggi in Core.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
There are a few issues/open points with Elements/Liquid signmessage and verifymessage
- Elements Core uses the same prefix as Bitcoin
- During verification the blinding key is ignored
- Verification only works for
p2pkh, see https://github.com/ElementsProject/elements/issues/976
Are we ok with all of the above? or do we we want to change something?
Do we still want to support message signing in Core?
Do we want to extend message signing and verification to non-p2pkh? Do we want to do it outside Core?
FWIW, here is a summary of what some existing wallets implement
| Core | GDK | Electrum | |||
|---|---|---|---|---|---|
| signmessage | bitcoin | p2pkh | ✅ | ✅ | ✅ |
| signmessage | bitcoin | p2sh-p2wpkh | ❌ | ✅ | ✅ |
| signmessage | bitcoin | p2wpkh | ❌ | ✅ | ✅ |
| verifymessage | bitcoin | p2pkh | ✅ | ❌ | ✅ |
| verifymessage | bitcoin | p2sh-p2wpkh | ❌ | ❌ | ✅ |
| verifymessage | bitcoin | p2wpkh | ❌ | ❌ | ✅ |
| signmessage | liquid | p2pkh | ✅ | ❌ | ❌ |
| signmessage | liquid | p2sh-p2wpkh | ❌ | ❌ | ❌ |
| signmessage | liquid | p2wpkh | ❌ | ❌ | ❌ |
| verifymessage | liquid | p2pkh | ✅ | ❌ | ❌ |
| verifymessage | liquid | p2sh-p2wpkh | ❌ | ❌ | ❌ |
| verifymessage | liquid | p2wpkh | ❌ | ❌ | ❌ |
Electrum was included in the above table since it was used as a reference to add support for non-p2pkh signing to GDK.
- Lingua principale
- C++
- Stelle
- 1.2k
- Fork
- 418
- Merge medio
- 1g 12h
- PR unite (30g)
- 13
Guida per i contributori
Apri la 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 ElementsProject/elements
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 58/100
ElementsProject/elements#1555 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
ElementsProject/elements#1521 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 32/100
ElementsProject/elements#1516 · 5 commenti ·
-
Compilation error on WSL Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
ElementsProject/elements#1502 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 38/100
ElementsProject/elements#1500 · 2 commenti ·
Tutte le issue di ElementsProject/elements
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
ros2/message_filters#338 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
subsurface/subsurface#4984 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
google-ai-edge/LiteRT-LM#3739 ·