Commit Graph
31401 Commits
Author SHA1 Message Date
duke 38699a7d47 Merge pull request 'Antispam defenses' (#322) from antispam into dev
Reviewed-on: https://git.hush.is/hush/hush3/pulls/322
2023-10-10 15:55:11 +00:00
Duke 84a0c2c35e antispam test 2023-10-07 14:16:59 -04:00
Duke b386cd1acf Scripts to test this branch 2023-10-07 14:15:12 -04:00
Duke c8a88e1168 Check null randomx dataset before calling randomx_dataset_item_count 2023-10-07 14:07:34 -04:00
Duke aa5cbee69c Remove dead code 2023-09-18 20:57:47 -04:00
Duke d7cbdcab28 Always log when skipping a zaddr during z_importwallet 2023-09-18 20:44:16 -04:00
Duke 2308db22ee Antispam defenses
This code is inspired by
https://github.com/PirateNetwork/pirate/commit/db292a49ddc13374f43b8c16217e171316be53d7
with various improvements that will be documented below.

The largest improvement is that this code will defend against a spammer using shielded inputs (zins)
or shielded outputs (zouts) while the Pirate code only defends against zout spam.

We wrote a new RPC called z_getstats to study exactly what the distribution of shielded inputs (zins)
and shielded outputs (zouts) look like on HUSH mainnet. Sietch will never make a ztx that contains
more than 9 zouts and so transactions with 10 or more zouts are extremely rare. They correspond to custom
transactions created via code or CLI or mining pool payouts. We allow at most one of these per block. If
there are two, one will remain in the mempool and be mined in the subsequent block. Our code is more strict,
as Pirate will allow up to 6 of these transactions in a single block.

Transactions with many shielded inputs do occur normally when users spend many small shielded unspent outputs
(zutxos) in one transaction, but we determined that a cutoff of 50 zins is quite rare. Between blocks
14000000 and 15000000 only 27 ztxs had 50 or more zins, which is 0.03% . We allow at most one of these
per block and if there are more, they will wait to be mined in a subsequent block.

Also note that a transaction can match both criteria of having large zins and large zouts, so for instance,
if there is a transaction with 50 zins and 10 zouts, it counts towards both requirements and no other
transactions with >=50 zins or >=10 zouts will be mined in that block.

If >=200 transactions with either large zins or large zouts are broadcast to the network it will take at least
200 blocks for them to be mined and so via existing rules for ztx expiration they will expire and be removed
from the mempool, since by default all ztxs expire after 200 blocks. Since normal ztxs that match these
criteria are very rare, the only case when this might happen is during a spam attack and so the attackers
transactions expiring is another part of these defenses.

Other improvements are that we log txids of transactions with large zins or zouts and we do not support a
command line option to turn this protection off. This forces a potential attacker to compile their own custom
code if they want to subvert these protections on their own node and blocks they mine.

Similar to Pirate, these changes are not consensus changes but may be made consensus requirements
in the future.

These protections are not specific to HUSH and are enabled for all HSC's, including DragonX.
2023-09-18 13:30:40 -04:00
Duke 09555fbee2 Allow abortrescan during RPC warmup
If we don't, we can get the hilarious error message that the node is Rescanning...
when trying to run abortrescan when the node automatically does a rescan on boot.
2023-09-18 12:43:27 -04:00
duke f2ae9a354a Merge pull request 'z_getstats RPC' (#321) from z_getstats into dev
Reviewed-on: https://git.hush.is/hush/hush3/pulls/321
2023-09-09 15:08:52 +00:00
Duke 7c45e66fbe Also return start and ending height in z_getstats json 2023-09-08 08:28:14 -04:00
Duke 4aca3493e3 Even more zstats
Example data for the entire history of the current HUSH mainnet :

  ./src/hush-cli z_getstats 1 1487622
{
  "total_ztxs": 414962,
  "total_zins": 798083,
  "total_zouts": 3312131,
  "total_ztxs_10_or_more_zins": 6789,
  "total_ztxs_25_or_more_zins": 1779,
  "total_ztxs_50_or_more_zins": 688,
  "total_ztxs_100_or_more_zins": 174,
  "total_ztxs_10_or_more_zouts": 2855,
  "total_ztxs_25_or_more_zouts": 394,
  "total_ztxs_50_or_more_zouts": 314,
  "total_ztxs_100_or_more_zouts": 208,
  "avg_zins": 1.923267672702561,
  "avg_zouts": 7.981769415030774,
  "largest_zins": 517,
  "largest_zins_txid": "69f126edd5a0189fbbe84b0824eb48e16eddf180e7d5d4f34c4296d0f868ac7f",
  "largest_zouts": 210,
  "largest_zouts_txid": "2a3155f73fab9191978e77e03be8ec7167372c4549113a6eb3f8a9d343f749ba"
}
2023-09-05 11:30:27 -04:00
Duke ff7a597032 Lots of more data for z_getstats 2023-09-05 00:35:11 -04:00
Duke 7ea88bb303 Return total zins+zouts in json 2023-09-04 09:03:06 -04:00
Duke 8eaba566fd Force avg zins/zouts to be a double 2023-09-04 08:57:23 -04:00
Duke 0f4956dcd5 Initialize variables in z_getstats correctly 2023-09-04 08:50:05 -04:00
Duke 96ae2d61ca z_getstats RPC that calculates various stats about ztxs in a block range 2023-09-04 08:22:12 -04:00
Duke 1c45a71b05 Remove dead code 2023-09-03 08:52:28 -04:00
Duke e2521ac2fa Remove unused sproutfunds argument 2023-09-03 08:48:06 -04:00
Duke bd38a12512 Remove useless sprout key from coinsupply RPC 2023-09-03 08:40:06 -04:00
Duke bacc08e817 Remove sprout data from valuePools
This codebase does not support sprout, the data will always be zero/empty
and is essentially useless cruft, so we delete it.
2023-09-03 08:37:19 -04:00
Duke a90f03ce6d Fix build-debian-package.sh docs and document how to make an aarch64 deb 2023-08-16 10:37:23 -04:00
duke c6859b676e Update 'doc/developer-notes.md' 2023-08-09 00:05:00 +00:00
Duke 4d3d880036 Add docs for adding a new PoW algo 2023-08-08 15:40:52 -04:00
fekt e53220b5f1 Save checkpoints as individual .json files 2023-07-09 11:49:55 -04:00
fekt 803e1d0f21 Use older version of cURL for Windows 2023-07-05 13:29:36 -04:00
Duke 8969a9d8fe Put binaries into a properly named subdirectory, fixes #311 2023-07-04 11:14:56 -04:00
fekt 2ba2122b26 Update relnotes 2023-07-02 11:26:54 -04:00
Duke 7017d8e06c Add info about DRAGONX checkpoints to release process 2023-06-30 06:42:33 -04:00
Duke d3cc17e9bd Update relnotes for 3.9.4 2023-06-30 06:37:37 -04:00
fekt 0ee828805e Update README 2023-06-29 23:39:33 -04:00
fekt 20ee425ece Updated checkpoints 2023-06-29 23:31:53 -04:00
fekt e86d43a813 Merge branch 'dev' of https://git.hush.is/hush/hush3 into dev 2023-06-29 22:28:49 -04:00
fekt 9e524663e2 Update manpages 2023-06-29 22:28:39 -04:00
fekt b7359ef70e Update chainparamsseeds.h 2023-06-29 21:16:46 -04:00
nullfekt 44595d5abe Add additional community seed node 2023-06-29 20:54:20 -04:00
nullfekt 27b2a49740 Update README.md 2023-06-29 20:38:09 -04:00
nullfekt 03071ef0cd Bump version 2023-06-29 20:23:28 -04:00
Duke 9508095186 Remove unused code 2023-06-29 11:48:30 -04:00
Duke 9d2720d118 Merge branch 'duke' into dev 2023-06-20 05:17:52 -07:00
Duke c933771f85 Merge branch 'danger' into duke 2023-06-17 21:40:36 -04:00
Duke fc69220e78 Remove unused code 2023-06-17 21:38:39 -04:00
Duke b03c8f3108 Remove unused code 2023-06-17 21:31:05 -04:00
Duke 31cc5ee901 Remove unused code 2023-06-17 21:22:41 -04:00
Duke 8f7e53a6fe Merge branch 'duke' into danger 2023-06-17 09:52:34 -04:00
Duke 5abc550af7 Dragonx has a blocktime of 36s not 60s 2023-06-17 09:52:12 -04:00
Duke e608600f07 Merge branch 'duke' into danger 2023-06-17 09:48:31 -04:00
Duke 97c7e814cf Add checkpoints for DRAGONX 2023-06-15 15:47:14 -04:00
Duke 1f0b538540 Take into account ac_name to calculate total txs 2023-06-15 15:43:08 -04:00
Duke f12bfeb7f4 Fix bug where it uses the wrong -ac_name 2023-06-15 15:38:16 -04:00
Duke b8ae39ccc7 Enable the option for dragonx checkpoints 2023-06-15 10:25:59 -04:00