zkbot
8a6c070bfa
Auto merge of #1870 - str4d:1749-benchmark-rescanning, r=str4d
...
Add benchmarks for rescan components
Part of #1749 .
2016-12-09 07:12:22 +00:00
zkbot
a398121503
Auto merge of #1919 - ebfull:abstract-verification, r=str4d
...
Isolate verification to a ProofVerifier context object that allows verification behavior to be tuned by the caller.
This is an alternative foundation for #1892 , i.e., #1892 will have to be changed if this PR is accepted.
I think this is a safer approach because it allows us to isolate verification behavior to a single object. This will come in handy when @arielgabizon finishes the batching code.
2016-12-09 06:22:53 +00:00
Jack Grigg
88b7f3c28b
Generate JS for trydecryptnotes, make number of addresses a variable
2016-12-09 16:59:39 +13:00
zkbot
e68e26c208
Auto merge of #1911 - bitcartel:1823_witness_does_not_have_same_anchor_as_change_input, r=str4d
...
With chained joinsplits, witness anchors for input notes no longer cross block boundaries
Closes #1823
2016-12-08 01:23:40 +00:00
Simon
84e8c5f921
WitnessAnchorData only needs to store one witness per JSOutPoint.
2016-12-07 12:05:29 -08:00
Jack Grigg
d85758f5cc
Document behaviour of CWallet::SetBestChain
2016-12-07 19:30:30 +13:00
Sean Bowe
bc59f53722
Isolate verification to a ProofVerifier context object that allows verification behavior to be tuned by the caller.
2016-12-05 12:40:06 -07:00
Simon
2d931e905b
Fixes #1823 . Witness anchors for input notes no longer cross block boundaries.
2016-12-03 00:22:42 -08:00
Simon
af53da0225
Closes #1903 . Add fee parameter to z_sendmany.
2016-11-30 14:23:35 -08:00
Jack Grigg
03f83b9b0d
Write witness caches when writing the best block
...
For steady-state operation, this reduces the average time between wallet disk
writes from once per block to once per hour.
On -rescan, witness caches are only written out at the end along with the best
block, increasing speed while ensuring that on-disk state is kept consistent.
Witness caches are now never recreated during a -reindex, on the assumption that
the blocks themselves are not changing (the chain is just being reconstructed),
and so the witnesses will remain valid.
Part of #1749 .
2016-11-30 14:04:37 +13:00
Jack Grigg
0bb3d40f90
Add benchmark for incrementing note witnesses
2016-11-17 22:26:46 +13:00
Jack Grigg
0fbab55b1b
Add benchmark for attempting decryption of notes
2016-11-17 19:09:45 +13:00
Simon
0afd3687c8
Closes #1854 . z_sendmany selects more utxos to avoid dust change output.
2016-11-15 17:12:37 -08:00
Simon
de38bfbbed
Track the correct change witness across chained joinsplits
2016-11-15 12:39:16 -08:00
Jack Grigg
a4ef3aa9b5
Clear witnessHeight and nWitnessCacheSize in ClearNoteWitnessCache
2016-11-15 17:53:41 +13:00
Jack Grigg
ccfd8aae1d
Add code comments about CNoteData::witnessHeight
2016-11-15 16:42:22 +13:00
Jack Grigg
4a6a48479d
Copy over CNoteData::witnessHeight when updating wallet tx
2016-11-15 14:08:41 +13:00
Jack Grigg
40ef121e6a
Correctly set CNoteData::witnessHeight when decrementing witness caches
...
Closes #1715
2016-11-15 14:08:35 +13:00
Simon
9384e74c50
Closes #1833 . Format currency amounts in z_sendmany error message.
...
Improve coverage of possible error states from z_sendmany.
Refactor qa test for z_sendmany operations.
2016-11-10 16:57:36 -08:00
ayleph
48f9c65b55
Correct spelling error in z_sendmany error output
...
Change "alllowed" -> "allowed."
2016-11-08 13:13:48 -08:00
Simon
3920292b43
Add transaction size and zaddr output limit checks to z_sendmany.
2016-11-07 09:29:56 -08:00
Simon
38276c6ba2
Add GenIdentity, an identity function for MappedShuffle.
...
We use this function in z_sendmany as part of the fix for #1779 .
2016-11-05 14:17:19 -07:00
Simon
7c463780cf
Fixes #1779 so that sending to multiple zaddrs no longer fails.
...
Commit 2eeb6b randomized the order of input and output notes,
but this is now known to prevent the chaining of multiple joinsplits
in a single transaction. The root cause has yet to be determined.
This patch is a temporary fix and disables the shuffling of input
and output notes. It also adds a chained joinsplit test to the
python qa test suite.
2016-11-04 23:23:48 -07:00
Sean Bowe
c4643bd949
Throw more descriptive exceptions when the constraint system is violated.
2016-11-01 14:52:03 -06:00
Jack Grigg
1749dd69c7
Fix incorrect locking in CCryptoKeyStore
2016-10-31 19:57:30 -05:00
Sean Bowe
58c4c0bba3
Change some Bitcoin strings to Zcash strings.
2016-10-26 13:29:05 -06:00
zkbot
489f1d38ee
Auto merge of #1629 - bitcartel:1616_wallet_disallow_coinbase_change, r=daira
...
Add z_sendmany rule that when coinbase utxos must be spent in their entirety, with no change
See #1616 - coinbase utxos must be consumed entirely, without any change, since there
is currently no way to specify a change address in z_sendmany.
2016-10-25 15:40:36 -04:00
Simon
ad0ba9467d
Add assert to AsyncRPCOperation_sendmany
2016-10-25 11:25:05 -07:00
Simon
db026807b8
Add z_sendmany rule that when sending coinbase utxos to a zaddr
...
they must be consumed entirely, without any change, since there
is currently no way to specify a change address in z_sendmany.
2016-10-24 21:34:16 -07:00
Simon
10d1b90987
Fix incorrect error message in z_sendmany
2016-10-24 07:59:32 -07:00
zkbot
71043784dd
Auto merge of #1586 - bitcartel:z_sendmany_amount_too_strict, r=daira
...
Fix issue where z_sendmany amount parameter is too strict
Upstream treats an amount parameter of `1` the same as `1.0`. Third-party would like this fixed.
2016-10-21 22:10:41 -04:00
zkbot
fd7ef7b632
Auto merge of #1584 - bitcartel:1504_zca_011_logging, r=daira
...
More granular control over logging of z_* calls
Closes #1504
2016-10-21 21:32:46 -04:00
zkbot
cd8e856df7
Auto merge of #1585 - bitcartel:1583_stop_test_writing_to_testnet3, r=daira
...
Closes #1583 by setting up the datadir for the wallet gtest.
Stops test writing to ~/.zcash/testnet3.
Also related to #1506
2016-10-21 03:15:00 -04:00
zkbot
323b8214c2
Auto merge of #1580 - str4d:1378-repair-witness-cache, r=daira
...
Clear witness cache when re-witnessing notes
Closes #1378
2016-10-21 00:24:39 -04:00
Simon
75407c6872
Fix issue where z_sendmany is too strict and does not allow integer based amount e.g. 1 which is the same as 1.0
2016-10-20 17:03:57 -07:00
Simon
ffd2cb3437
Closes #1583 by setting up the datadir for the wallet gtest.
2016-10-20 16:45:02 -07:00
Simon
446f867b4c
Add debug option "zrpcunsafe" to be used when logging more sensitive information such as the memo field of a note.
2016-10-20 15:24:31 -07:00
zkbot
f90526579b
Auto merge of #1569 - str4d:1552-disable-wallet-encryption, r=daira
...
Disable wallet encryption
Closes #1552
2016-10-20 17:01:51 -04:00
Jack Grigg
1532cb75f3
Throw an error when encryptwallet is disabled
2016-10-20 12:58:20 -05:00
Jack Grigg
1b407cba2b
Add heights to log output
2016-10-20 11:02:15 -05:00
Jack Grigg
878c4b1b50
Clear witness cache when re-witnessing notes
...
Closes #1378
2016-10-20 10:32:20 -05:00
Jack Grigg
aa36398b27
Add more assertions, throw if find_output params are invalid
2016-10-19 08:49:08 -05:00
Jack Grigg
62c0aa9e58
Disable wallet encryption
...
Closes #1552
2016-10-18 17:47:17 -05:00
Jack Grigg
328d39d29c
Remove stale comment
2016-10-18 13:10:20 -05:00
Jack Grigg
3774c944f8
Refactor test code to better test JSDescription::Randomized()
2016-10-18 12:44:56 -05:00
Jack Grigg
2eeb6bebde
Randomize JoinSplits in z_sendmany
2016-10-18 10:41:30 -05:00
Jack Grigg
b6961fc112
Increment witnesses for new transactions on rescan
...
Closes #1400
2016-10-17 09:55:02 -05:00
Daira Hopwood
95e32d88b0
Add another assertion about the witness cache.
...
Signed-off-by: Daira Hopwood <daira@jacaranda.org >
2016-10-17 05:09:27 +01:00
Daira Hopwood
8e41408aa7
Add another assertion to narrow down where the bug occurs.
...
Signed-off-by: Daira Hopwood <daira@jacaranda.org >
2016-10-17 05:09:27 +01:00
Daira Hopwood
8a7d37befd
Move the increment of nWitnessCacheSize to make the later assertions correct.
...
Signed-off-by: Daira Hopwood <daira@jacaranda.org >
2016-10-17 05:09:27 +01:00