mirror of
https://github.com/bitcoin/bitcoin.git
synced 2024-11-20 10:38:42 +01:00
Add SegWit support to importmulti with some ProcessImport cleanup
This commit is contained in:
parent
2a2cac7873
commit
f6ed748cf0
@ -809,29 +809,24 @@ UniValue dumpwallet(const JSONRPCRequest& request)
|
|||||||
static UniValue ProcessImport(CWallet * const pwallet, const UniValue& data, const int64_t timestamp) EXCLUSIVE_LOCKS_REQUIRED(pwallet->cs_wallet)
|
static UniValue ProcessImport(CWallet * const pwallet, const UniValue& data, const int64_t timestamp) EXCLUSIVE_LOCKS_REQUIRED(pwallet->cs_wallet)
|
||||||
{
|
{
|
||||||
try {
|
try {
|
||||||
bool success = false;
|
// First ensure scriptPubKey has either a script or JSON with "address" string
|
||||||
|
|
||||||
// Required fields.
|
|
||||||
const UniValue& scriptPubKey = data["scriptPubKey"];
|
const UniValue& scriptPubKey = data["scriptPubKey"];
|
||||||
|
bool isScript = scriptPubKey.getType() == UniValue::VSTR;
|
||||||
// Should have script or JSON with "address".
|
if (!isScript && !(scriptPubKey.getType() == UniValue::VOBJ && scriptPubKey.exists("address"))) {
|
||||||
if (!(scriptPubKey.getType() == UniValue::VOBJ && scriptPubKey.exists("address")) && !(scriptPubKey.getType() == UniValue::VSTR)) {
|
throw JSONRPCError(RPC_INVALID_PARAMETER, "scriptPubKey must be string with script or JSON with address string");
|
||||||
throw JSONRPCError(RPC_INVALID_PARAMETER, "Invalid scriptPubKey");
|
|
||||||
}
|
}
|
||||||
|
const std::string& output = isScript ? scriptPubKey.get_str() : scriptPubKey["address"].get_str();
|
||||||
|
|
||||||
// Optional fields.
|
// Optional fields.
|
||||||
const std::string& strRedeemScript = data.exists("redeemscript") ? data["redeemscript"].get_str() : "";
|
const std::string& strRedeemScript = data.exists("redeemscript") ? data["redeemscript"].get_str() : "";
|
||||||
|
const std::string& witness_script_hex = data.exists("witnessscript") ? data["witnessscript"].get_str() : "";
|
||||||
const UniValue& pubKeys = data.exists("pubkeys") ? data["pubkeys"].get_array() : UniValue();
|
const UniValue& pubKeys = data.exists("pubkeys") ? data["pubkeys"].get_array() : UniValue();
|
||||||
const UniValue& keys = data.exists("keys") ? data["keys"].get_array() : UniValue();
|
const UniValue& keys = data.exists("keys") ? data["keys"].get_array() : UniValue();
|
||||||
const bool internal = data.exists("internal") ? data["internal"].get_bool() : false;
|
const bool internal = data.exists("internal") ? data["internal"].get_bool() : false;
|
||||||
const bool watchOnly = data.exists("watchonly") ? data["watchonly"].get_bool() : false;
|
const bool watchOnly = data.exists("watchonly") ? data["watchonly"].get_bool() : false;
|
||||||
const std::string& label = data.exists("label") && !internal ? data["label"].get_str() : "";
|
const std::string& label = data.exists("label") ? data["label"].get_str() : "";
|
||||||
|
|
||||||
bool isScript = scriptPubKey.getType() == UniValue::VSTR;
|
// Generate the script and destination for the scriptPubKey provided
|
||||||
bool isP2SH = strRedeemScript.length() > 0;
|
|
||||||
const std::string& output = isScript ? scriptPubKey.get_str() : scriptPubKey["address"].get_str();
|
|
||||||
|
|
||||||
// Parse the output.
|
|
||||||
CScript script;
|
CScript script;
|
||||||
CTxDestination dest;
|
CTxDestination dest;
|
||||||
|
|
||||||
@ -855,35 +850,38 @@ static UniValue ProcessImport(CWallet * const pwallet, const UniValue& data, con
|
|||||||
|
|
||||||
// Watchonly and private keys
|
// Watchonly and private keys
|
||||||
if (watchOnly && keys.size()) {
|
if (watchOnly && keys.size()) {
|
||||||
throw JSONRPCError(RPC_INVALID_PARAMETER, "Incompatibility found between watchonly and keys");
|
throw JSONRPCError(RPC_INVALID_PARAMETER, "Watch-only addresses should not include private keys");
|
||||||
}
|
}
|
||||||
|
|
||||||
// Internal + Label
|
// Internal addresses should not have a label
|
||||||
if (internal && data.exists("label")) {
|
if (internal && data.exists("label")) {
|
||||||
throw JSONRPCError(RPC_INVALID_PARAMETER, "Incompatibility found between internal and label");
|
throw JSONRPCError(RPC_INVALID_PARAMETER, "Internal addresses should not have a label");
|
||||||
}
|
}
|
||||||
|
|
||||||
// Keys / PubKeys size check.
|
// Force users to provide the witness script in its field rather than redeemscript
|
||||||
if (!isP2SH && (keys.size() > 1 || pubKeys.size() > 1)) { // Address / scriptPubKey
|
if (!strRedeemScript.empty() && script.IsPayToWitnessScriptHash()) {
|
||||||
throw JSONRPCError(RPC_INVALID_PARAMETER, "More than private key given for one address");
|
throw JSONRPCError(RPC_INVALID_PARAMETER, "P2WSH addresses have an empty redeemscript. Please provide the witnessscript instead.");
|
||||||
}
|
}
|
||||||
|
|
||||||
// Invalid P2SH redeemScript
|
CScript scriptpubkey_script = script;
|
||||||
if (isP2SH && !IsHex(strRedeemScript)) {
|
CTxDestination scriptpubkey_dest = dest;
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Invalid redeem script");
|
bool allow_p2wpkh = true;
|
||||||
}
|
|
||||||
|
|
||||||
// Process. //
|
|
||||||
|
|
||||||
// P2SH
|
// P2SH
|
||||||
if (isP2SH) {
|
if (!strRedeemScript.empty() && script.IsPayToScriptHash()) {
|
||||||
|
// Check the redeemScript is valid
|
||||||
|
if (!IsHex(strRedeemScript)) {
|
||||||
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Invalid redeem script: must be hex string");
|
||||||
|
}
|
||||||
|
|
||||||
// Import redeem script.
|
// Import redeem script.
|
||||||
std::vector<unsigned char> vData(ParseHex(strRedeemScript));
|
std::vector<unsigned char> vData(ParseHex(strRedeemScript));
|
||||||
CScript redeemScript = CScript(vData.begin(), vData.end());
|
CScript redeemScript = CScript(vData.begin(), vData.end());
|
||||||
|
CScriptID redeem_id(redeemScript);
|
||||||
|
|
||||||
// Invalid P2SH address
|
// Check that the redeemScript and scriptPubKey match
|
||||||
if (!script.IsPayToScriptHash()) {
|
if (GetScriptForDestination(redeem_id) != script) {
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Invalid P2SH address / script");
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "The redeemScript does not match the scriptPubKey");
|
||||||
}
|
}
|
||||||
|
|
||||||
pwallet->MarkDirty();
|
pwallet->MarkDirty();
|
||||||
@ -892,103 +890,83 @@ static UniValue ProcessImport(CWallet * const pwallet, const UniValue& data, con
|
|||||||
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
||||||
}
|
}
|
||||||
|
|
||||||
CScriptID redeem_id(redeemScript);
|
|
||||||
if (!pwallet->HaveCScript(redeem_id) && !pwallet->AddCScript(redeemScript)) {
|
if (!pwallet->HaveCScript(redeem_id) && !pwallet->AddCScript(redeemScript)) {
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding p2sh redeemScript to wallet");
|
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding p2sh redeemScript to wallet");
|
||||||
}
|
}
|
||||||
|
|
||||||
CScript redeemDestination = GetScriptForDestination(redeem_id);
|
// Now set script to the redeemScript so we parse the inner script as P2WSH or P2WPKH below
|
||||||
|
script = redeemScript;
|
||||||
|
ExtractDestination(script, dest);
|
||||||
|
}
|
||||||
|
|
||||||
if (::IsMine(*pwallet, redeemDestination) == ISMINE_SPENDABLE) {
|
// (P2SH-)P2WSH
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "The wallet already contains the private key for this address or script");
|
if (!witness_script_hex.empty() && script.IsPayToWitnessScriptHash()) {
|
||||||
|
if (!IsHex(witness_script_hex)) {
|
||||||
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Invalid witness script: must be hex string");
|
||||||
}
|
}
|
||||||
|
|
||||||
pwallet->MarkDirty();
|
// Generate the scripts
|
||||||
|
std::vector<unsigned char> witness_script_parsed(ParseHex(witness_script_hex));
|
||||||
|
CScript witness_script = CScript(witness_script_parsed.begin(), witness_script_parsed.end());
|
||||||
|
CScriptID witness_id(witness_script);
|
||||||
|
|
||||||
if (!pwallet->AddWatchOnly(redeemDestination, timestamp)) {
|
// Check that the witnessScript and scriptPubKey match
|
||||||
|
if (GetScriptForDestination(WitnessV0ScriptHash(witness_script)) != script) {
|
||||||
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "The witnessScript does not match the scriptPubKey or redeemScript");
|
||||||
|
}
|
||||||
|
|
||||||
|
// Add the witness script as watch only only if it is not for P2SH-P2WSH
|
||||||
|
if (!scriptpubkey_script.IsPayToScriptHash() && !pwallet->AddWatchOnly(witness_script, timestamp)) {
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
||||||
}
|
}
|
||||||
|
|
||||||
// add to address book or update label
|
if (!pwallet->HaveCScript(witness_id) && !pwallet->AddCScript(witness_script)) {
|
||||||
if (IsValidDestination(dest)) {
|
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding p2wsh witnessScript to wallet");
|
||||||
pwallet->SetAddressBook(dest, label, "receive");
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// Import private keys.
|
// Now set script to the witnessScript so we parse the inner script as P2PK or P2PKH below
|
||||||
|
script = witness_script;
|
||||||
|
ExtractDestination(script, dest);
|
||||||
|
allow_p2wpkh = false; // P2WPKH cannot be embedded in P2WSH
|
||||||
|
}
|
||||||
|
|
||||||
|
// (P2SH-)P2PK/P2PKH/P2WPKH
|
||||||
|
if (dest.type() == typeid(CKeyID) || dest.type() == typeid(WitnessV0KeyHash)) {
|
||||||
|
if (!allow_p2wpkh && dest.type() == typeid(WitnessV0KeyHash)) {
|
||||||
|
throw JSONRPCError(RPC_INVALID_PARAMETER, "P2WPKH cannot be embedded in P2WSH");
|
||||||
|
}
|
||||||
|
if (keys.size() > 1 || pubKeys.size() > 1) {
|
||||||
|
throw JSONRPCError(RPC_INVALID_PARAMETER, "More than one key given for one single-key address");
|
||||||
|
}
|
||||||
|
CPubKey pubkey;
|
||||||
if (keys.size()) {
|
if (keys.size()) {
|
||||||
for (size_t i = 0; i < keys.size(); i++) {
|
pubkey = DecodeSecret(keys[0].get_str()).GetPubKey();
|
||||||
const std::string& privkey = keys[i].get_str();
|
|
||||||
|
|
||||||
CKey key = DecodeSecret(privkey);
|
|
||||||
|
|
||||||
if (!key.IsValid()) {
|
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Invalid private key encoding");
|
|
||||||
}
|
|
||||||
|
|
||||||
CPubKey pubkey = key.GetPubKey();
|
|
||||||
assert(key.VerifyPubKey(pubkey));
|
|
||||||
|
|
||||||
CKeyID vchAddress = pubkey.GetID();
|
|
||||||
pwallet->MarkDirty();
|
|
||||||
pwallet->SetAddressBook(vchAddress, label, "receive");
|
|
||||||
|
|
||||||
if (pwallet->HaveKey(vchAddress)) {
|
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Already have this key");
|
|
||||||
}
|
|
||||||
|
|
||||||
pwallet->mapKeyMetadata[vchAddress].nCreateTime = timestamp;
|
|
||||||
|
|
||||||
if (!pwallet->AddKeyPubKey(key, pubkey)) {
|
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding key to wallet");
|
|
||||||
}
|
|
||||||
|
|
||||||
pwallet->UpdateTimeFirstKey(timestamp);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
if (pubKeys.size()) {
|
||||||
success = true;
|
|
||||||
} else {
|
|
||||||
// Import public keys.
|
|
||||||
if (pubKeys.size() && keys.size() == 0) {
|
|
||||||
const std::string& strPubKey = pubKeys[0].get_str();
|
const std::string& strPubKey = pubKeys[0].get_str();
|
||||||
|
|
||||||
if (!IsHex(strPubKey)) {
|
if (!IsHex(strPubKey)) {
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Pubkey must be a hex string");
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Pubkey must be a hex string");
|
||||||
}
|
}
|
||||||
|
std::vector<unsigned char> vData(ParseHex(pubKeys[0].get_str()));
|
||||||
std::vector<unsigned char> vData(ParseHex(strPubKey));
|
CPubKey pubkey_temp(vData.begin(), vData.end());
|
||||||
CPubKey pubKey(vData.begin(), vData.end());
|
if (pubkey.size() && pubkey_temp != pubkey) {
|
||||||
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Private key does not match public key for address");
|
||||||
if (!pubKey.IsFullyValid()) {
|
}
|
||||||
|
pubkey = pubkey_temp;
|
||||||
|
}
|
||||||
|
if (pubkey.size() > 0) {
|
||||||
|
if (!pubkey.IsFullyValid()) {
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Pubkey is not a valid public key");
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Pubkey is not a valid public key");
|
||||||
}
|
}
|
||||||
|
|
||||||
CTxDestination pubkey_dest = pubKey.GetID();
|
// Check the key corresponds to the destination given
|
||||||
|
std::vector<CTxDestination> destinations = GetAllDestinationsForKey(pubkey);
|
||||||
// Consistency check.
|
if (std::find(destinations.begin(), destinations.end(), dest) == destinations.end()) {
|
||||||
if (!(pubkey_dest == dest)) {
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Key does not match address destination");
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Consistency check failed");
|
|
||||||
}
|
}
|
||||||
|
|
||||||
CScript pubKeyScript = GetScriptForDestination(pubkey_dest);
|
// This is necessary to force the wallet to import the pubKey
|
||||||
|
CScript scriptRawPubKey = GetScriptForRawPubKey(pubkey);
|
||||||
if (::IsMine(*pwallet, pubKeyScript) == ISMINE_SPENDABLE) {
|
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "The wallet already contains the private key for this address or script");
|
|
||||||
}
|
|
||||||
|
|
||||||
pwallet->MarkDirty();
|
|
||||||
|
|
||||||
if (!pwallet->AddWatchOnly(pubKeyScript, timestamp)) {
|
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
|
||||||
}
|
|
||||||
|
|
||||||
// add to address book or update label
|
|
||||||
if (IsValidDestination(pubkey_dest)) {
|
|
||||||
pwallet->SetAddressBook(pubkey_dest, label, "receive");
|
|
||||||
}
|
|
||||||
|
|
||||||
// TODO Is this necessary?
|
|
||||||
CScript scriptRawPubKey = GetScriptForRawPubKey(pubKey);
|
|
||||||
|
|
||||||
if (::IsMine(*pwallet, scriptRawPubKey) == ISMINE_SPENDABLE) {
|
if (::IsMine(*pwallet, scriptRawPubKey) == ISMINE_SPENDABLE) {
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "The wallet already contains the private key for this address or script");
|
throw JSONRPCError(RPC_WALLET_ERROR, "The wallet already contains the private key for this address or script");
|
||||||
@ -999,73 +977,61 @@ static UniValue ProcessImport(CWallet * const pwallet, const UniValue& data, con
|
|||||||
if (!pwallet->AddWatchOnly(scriptRawPubKey, timestamp)) {
|
if (!pwallet->AddWatchOnly(scriptRawPubKey, timestamp)) {
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
||||||
}
|
}
|
||||||
|
|
||||||
success = true;
|
|
||||||
}
|
|
||||||
|
|
||||||
// Import private keys.
|
|
||||||
if (keys.size()) {
|
|
||||||
const std::string& strPrivkey = keys[0].get_str();
|
|
||||||
|
|
||||||
// Checks.
|
|
||||||
CKey key = DecodeSecret(strPrivkey);
|
|
||||||
|
|
||||||
if (!key.IsValid()) {
|
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Invalid private key encoding");
|
|
||||||
}
|
|
||||||
|
|
||||||
CPubKey pubKey = key.GetPubKey();
|
|
||||||
assert(key.VerifyPubKey(pubKey));
|
|
||||||
|
|
||||||
CTxDestination pubkey_dest = pubKey.GetID();
|
|
||||||
|
|
||||||
// Consistency check.
|
|
||||||
if (!(pubkey_dest == dest)) {
|
|
||||||
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Consistency check failed");
|
|
||||||
}
|
|
||||||
|
|
||||||
CKeyID vchAddress = pubKey.GetID();
|
|
||||||
pwallet->MarkDirty();
|
|
||||||
pwallet->SetAddressBook(vchAddress, label, "receive");
|
|
||||||
|
|
||||||
if (pwallet->HaveKey(vchAddress)) {
|
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "The wallet already contains the private key for this address or script");
|
|
||||||
}
|
|
||||||
|
|
||||||
pwallet->mapKeyMetadata[vchAddress].nCreateTime = timestamp;
|
|
||||||
|
|
||||||
if (!pwallet->AddKeyPubKey(key, pubKey)) {
|
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding key to wallet");
|
|
||||||
}
|
|
||||||
|
|
||||||
pwallet->UpdateTimeFirstKey(timestamp);
|
|
||||||
|
|
||||||
success = true;
|
|
||||||
}
|
|
||||||
|
|
||||||
// Import scriptPubKey only.
|
|
||||||
if (pubKeys.size() == 0 && keys.size() == 0) {
|
|
||||||
if (::IsMine(*pwallet, script) == ISMINE_SPENDABLE) {
|
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "The wallet already contains the private key for this address or script");
|
|
||||||
}
|
|
||||||
|
|
||||||
pwallet->MarkDirty();
|
|
||||||
|
|
||||||
if (!pwallet->AddWatchOnly(script, timestamp)) {
|
|
||||||
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
|
||||||
}
|
|
||||||
|
|
||||||
// add to address book or update label
|
|
||||||
if (IsValidDestination(dest)) {
|
|
||||||
pwallet->SetAddressBook(dest, label, "receive");
|
|
||||||
}
|
|
||||||
|
|
||||||
success = true;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Import the address
|
||||||
|
if (::IsMine(*pwallet, scriptpubkey_script) == ISMINE_SPENDABLE) {
|
||||||
|
throw JSONRPCError(RPC_WALLET_ERROR, "The wallet already contains the private key for this address or script");
|
||||||
|
}
|
||||||
|
|
||||||
|
pwallet->MarkDirty();
|
||||||
|
|
||||||
|
if (!pwallet->AddWatchOnly(scriptpubkey_script, timestamp)) {
|
||||||
|
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding address to wallet");
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!watchOnly && !pwallet->HaveCScript(CScriptID(scriptpubkey_script)) && !pwallet->AddCScript(scriptpubkey_script)) {
|
||||||
|
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding scriptPubKey script to wallet");
|
||||||
|
}
|
||||||
|
|
||||||
|
// add to address book or update label
|
||||||
|
if (IsValidDestination(scriptpubkey_dest)) {
|
||||||
|
pwallet->SetAddressBook(scriptpubkey_dest, label, "receive");
|
||||||
|
}
|
||||||
|
|
||||||
|
// Import private keys.
|
||||||
|
for (size_t i = 0; i < keys.size(); i++) {
|
||||||
|
const std::string& strPrivkey = keys[i].get_str();
|
||||||
|
|
||||||
|
// Checks.
|
||||||
|
CKey key = DecodeSecret(strPrivkey);
|
||||||
|
|
||||||
|
if (!key.IsValid()) {
|
||||||
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Invalid private key encoding");
|
||||||
|
}
|
||||||
|
|
||||||
|
CPubKey pubKey = key.GetPubKey();
|
||||||
|
assert(key.VerifyPubKey(pubKey));
|
||||||
|
|
||||||
|
CKeyID vchAddress = pubKey.GetID();
|
||||||
|
pwallet->MarkDirty();
|
||||||
|
|
||||||
|
if (pwallet->HaveKey(vchAddress)) {
|
||||||
|
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Already have this key");
|
||||||
|
}
|
||||||
|
|
||||||
|
pwallet->mapKeyMetadata[vchAddress].nCreateTime = timestamp;
|
||||||
|
|
||||||
|
if (!pwallet->AddKeyPubKey(key, pubKey)) {
|
||||||
|
throw JSONRPCError(RPC_WALLET_ERROR, "Error adding key to wallet");
|
||||||
|
}
|
||||||
|
|
||||||
|
pwallet->UpdateTimeFirstKey(timestamp);
|
||||||
|
}
|
||||||
|
|
||||||
UniValue result = UniValue(UniValue::VOBJ);
|
UniValue result = UniValue(UniValue::VOBJ);
|
||||||
result.pushKV("success", UniValue(success));
|
result.pushKV("success", UniValue(true));
|
||||||
return result;
|
return result;
|
||||||
} catch (const UniValue& e) {
|
} catch (const UniValue& e) {
|
||||||
UniValue result = UniValue(UniValue::VOBJ);
|
UniValue result = UniValue(UniValue::VOBJ);
|
||||||
@ -1118,7 +1084,8 @@ UniValue importmulti(const JSONRPCRequest& mainRequest)
|
|||||||
" \"now\" can be specified to bypass scanning, for keys which are known to never have been used, and\n"
|
" \"now\" can be specified to bypass scanning, for keys which are known to never have been used, and\n"
|
||||||
" 0 can be specified to scan the entire blockchain. Blocks up to 2 hours before the earliest key\n"
|
" 0 can be specified to scan the entire blockchain. Blocks up to 2 hours before the earliest key\n"
|
||||||
" creation time of all keys being imported by the importmulti call will be scanned.\n"
|
" creation time of all keys being imported by the importmulti call will be scanned.\n"
|
||||||
" \"redeemscript\": \"<script>\" , (string, optional) Allowed only if the scriptPubKey is a P2SH address or a P2SH scriptPubKey\n"
|
" \"redeemscript\": \"<script>\" , (string, optional) Allowed only if the scriptPubKey is a P2SH or P2SH-P2WSH address/scriptPubKey\n"
|
||||||
|
" \"witnessscript\": \"<script>\" , (string, optional) Allowed only if the scriptPubKey is a P2SH-P2WSH or P2WSH address/scriptPubKey\n"
|
||||||
" \"pubkeys\": [\"<pubKey>\", ... ] , (array, optional) Array of strings giving pubkeys that must occur in the output or redeemscript\n"
|
" \"pubkeys\": [\"<pubKey>\", ... ] , (array, optional) Array of strings giving pubkeys that must occur in the output or redeemscript\n"
|
||||||
" \"keys\": [\"<key>\", ... ] , (array, optional) Array of strings giving private keys whose corresponding public keys must occur in the output or redeemscript\n"
|
" \"keys\": [\"<key>\", ... ] , (array, optional) Array of strings giving private keys whose corresponding public keys must occur in the output or redeemscript\n"
|
||||||
" \"internal\": <true> , (boolean, optional, default: false) Stating whether matching outputs should be treated as not incoming payments\n"
|
" \"internal\": <true> , (boolean, optional, default: false) Stating whether matching outputs should be treated as not incoming payments\n"
|
||||||
|
@ -198,7 +198,7 @@ class ImportMultiTest(BitcoinTestFramework):
|
|||||||
}])
|
}])
|
||||||
assert_equal(result[0]['success'], False)
|
assert_equal(result[0]['success'], False)
|
||||||
assert_equal(result[0]['error']['code'], -8)
|
assert_equal(result[0]['error']['code'], -8)
|
||||||
assert_equal(result[0]['error']['message'], 'Incompatibility found between watchonly and keys')
|
assert_equal(result[0]['error']['message'], 'Watch-only addresses should not include private keys')
|
||||||
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
||||||
assert_equal(address_assert['iswatchonly'], False)
|
assert_equal(address_assert['iswatchonly'], False)
|
||||||
assert_equal(address_assert['ismine'], False)
|
assert_equal(address_assert['ismine'], False)
|
||||||
@ -339,7 +339,7 @@ class ImportMultiTest(BitcoinTestFramework):
|
|||||||
}])
|
}])
|
||||||
assert_equal(result[0]['success'], False)
|
assert_equal(result[0]['success'], False)
|
||||||
assert_equal(result[0]['error']['code'], -8)
|
assert_equal(result[0]['error']['code'], -8)
|
||||||
assert_equal(result[0]['error']['message'], 'Incompatibility found between watchonly and keys')
|
assert_equal(result[0]['error']['message'], 'Watch-only addresses should not include private keys')
|
||||||
|
|
||||||
|
|
||||||
# Address + Public key + !Internal + Wrong pubkey
|
# Address + Public key + !Internal + Wrong pubkey
|
||||||
@ -355,7 +355,7 @@ class ImportMultiTest(BitcoinTestFramework):
|
|||||||
}])
|
}])
|
||||||
assert_equal(result[0]['success'], False)
|
assert_equal(result[0]['success'], False)
|
||||||
assert_equal(result[0]['error']['code'], -5)
|
assert_equal(result[0]['error']['code'], -5)
|
||||||
assert_equal(result[0]['error']['message'], 'Consistency check failed')
|
assert_equal(result[0]['error']['message'], 'Key does not match address destination')
|
||||||
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
||||||
assert_equal(address_assert['iswatchonly'], False)
|
assert_equal(address_assert['iswatchonly'], False)
|
||||||
assert_equal(address_assert['ismine'], False)
|
assert_equal(address_assert['ismine'], False)
|
||||||
@ -375,7 +375,7 @@ class ImportMultiTest(BitcoinTestFramework):
|
|||||||
result = self.nodes[1].importmulti(request)
|
result = self.nodes[1].importmulti(request)
|
||||||
assert_equal(result[0]['success'], False)
|
assert_equal(result[0]['success'], False)
|
||||||
assert_equal(result[0]['error']['code'], -5)
|
assert_equal(result[0]['error']['code'], -5)
|
||||||
assert_equal(result[0]['error']['message'], 'Consistency check failed')
|
assert_equal(result[0]['error']['message'], 'Key does not match address destination')
|
||||||
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
||||||
assert_equal(address_assert['iswatchonly'], False)
|
assert_equal(address_assert['iswatchonly'], False)
|
||||||
assert_equal(address_assert['ismine'], False)
|
assert_equal(address_assert['ismine'], False)
|
||||||
@ -395,7 +395,7 @@ class ImportMultiTest(BitcoinTestFramework):
|
|||||||
}])
|
}])
|
||||||
assert_equal(result[0]['success'], False)
|
assert_equal(result[0]['success'], False)
|
||||||
assert_equal(result[0]['error']['code'], -5)
|
assert_equal(result[0]['error']['code'], -5)
|
||||||
assert_equal(result[0]['error']['message'], 'Consistency check failed')
|
assert_equal(result[0]['error']['message'], 'Key does not match address destination')
|
||||||
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
||||||
assert_equal(address_assert['iswatchonly'], False)
|
assert_equal(address_assert['iswatchonly'], False)
|
||||||
assert_equal(address_assert['ismine'], False)
|
assert_equal(address_assert['ismine'], False)
|
||||||
@ -414,7 +414,7 @@ class ImportMultiTest(BitcoinTestFramework):
|
|||||||
}])
|
}])
|
||||||
assert_equal(result[0]['success'], False)
|
assert_equal(result[0]['success'], False)
|
||||||
assert_equal(result[0]['error']['code'], -5)
|
assert_equal(result[0]['error']['code'], -5)
|
||||||
assert_equal(result[0]['error']['message'], 'Consistency check failed')
|
assert_equal(result[0]['error']['message'], 'Key does not match address destination')
|
||||||
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
address_assert = self.nodes[1].getaddressinfo(address['address'])
|
||||||
assert_equal(address_assert['iswatchonly'], False)
|
assert_equal(address_assert['iswatchonly'], False)
|
||||||
assert_equal(address_assert['ismine'], False)
|
assert_equal(address_assert['ismine'], False)
|
||||||
|
Loading…
Reference in New Issue
Block a user