Labels Milestones

Implement zsweep #195

Merged

duke merged 49 commits from zsweep into dev 2 years ago

Conversation 0 Commits 49 Files Changed 28

duke commented 2 years ago

Owner

We have a few issues related to improvements, but core functionality seems to be working.

We have a few issues related to improvements, but core functionality seems to be working.

duke added 36 commits 2 years ago

c6e5b07a59 You might be a king or a little street zsweeper, but sooner or later you will dance with the reaper

02ef714901 Commit forgotten files

4aedeb557b Add z_sweepstatus RPC

4eec81b191 Fix some issues in z_sweepstatus

3730175971 Add some logging when skipping sweep/consolidation

0e3889c96a List z_sweepstatus in RPC list

1748f0f2a3 Improved logging and error checking in consolidation and sweeping

702c2163cd Log if we have no zsweepaddress or an invalid zsweepaddress

b4b6988eb0 Log when we use fee=0 for small inputs

b50c51f797 Log value of status and correctly return status value instead of always returning success, derp

57baf25b02 Do not add to amountConsolidated until we successfully create the tx and only do it once

88263995c8 Do not coredump if pnode=NULL in RelayTransaction

26fd16cb52 Avoid relaying expired transactions which causes other nodes to ban us

0865579e54 Merge remote-tracking branch 'origin/zsweep' into zsweep

4573a8336f Fix compile error

63f9e22308 log the function name correctly

3327b25a28 fix logging bug

ed1939929b fix compile issue

5c8c7507ec Add z_consolidationstatus RPC

9c5e5c550c Fix compile issues in z_consolidationstatus

55703fdb95 Do not ban nodes which relay expired transactions

456cf77b65 Exit RelayTransaction early if we have no nodes to relay to

8581012163 Sweep operations might not create any txs, so make logging more precise

8d8f988a19 Delete expired txs from the wallet, since they can never be included in a block

61ecffd074 Change default sweep interval to 10 and consolidation to be deterministic every 45 blocks

8770fb7035 Hopefully avoid a coredump when consolidation=1 and consolidatesaplingaddress is not specified

4647bc8525 Try to generate a stacktrace for runtime errors in consolidation

8f49549a1d Fix stacktrace

85f7ac4c8c Enable function names and line numbers in backtraces

1259dd512d Fix 'not enough conversion params' bug

772160af6d Add more logging for consolidation + sweeping

2a1cad1e5f Fix bug in z_sweepstatus

9347b1fa12 Sweep Interval should be 10 to match other defaults

5b15d790f9 Parse sweep+consolidation intervals on startup, add consolidationinterval config option, change consolidation interval default to 25, only validate consolidatesaplingaddress if consolidation is enabled

bd9006623c Avoid coredumps in ReacceptWalletTransactions and log the situation, which is possibly caused by corrupt wallet tx data

bd4d2d71d3 Merge branch 'dev' into zsweep

jahway603 added 1 commit 2 years ago

78f5021cc2 starting to support zsweepexclude functionality

jahway603 added 1 commit 2 years ago

ca3bb90e0c skip excluded address from zsweep

jahway603 added 1 commit 2 years ago

0c22e26358 updated z_sweepstatus

duke added 1 commit 2 years ago

88d3f11664 Save excluded address to be used later

jahway603 added 1 commit 2 years ago

b687d5f76e debug

duke added 1 commit 2 years ago

908240d07f Make z_sweepstatus match up with config option name

duke added 1 commit 2 years ago

f90715192e Do not delete expired txs for now, it doesn't work correctly, and log more info about them

duke added 2 commits 2 years ago

77d7aaba96 Return an array of addresses to exclude from zsweep in zsweep_status

6b79b3e12c Merge branch 'zsweep' of https://git.hush.is/hush/hush3 into zsweep

duke added 1 commit 2 years ago

f796d5d14e Support multiple zsweepexclude zaddrs

duke added 1 commit 2 years ago

12fa3013ce Fix definition of sweepExcludeAddresses

duke added 1 commit 2 years ago

35f376c8b5 Fix bugs in excluding multiple zsweep zaddrs and report all excluded zsweep zaddrs in z_sweepstatus

duke added 1 commit 2 years ago

62c9bce6f7 Correct error messsage about invalid zsweepexclude address

duke merged commit 61999b89ed into dev 2 years ago

duke referenced this issue from a commit 2 years ago

Merge pull request 'Implement zsweep' (#195) from zsweep into dev

The pull request has been merged as 61999b89ed.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

0-500 bounty
bounty up to 500 HUSH 2001-5000 bounty
bounty between 2001 and 5000 HUSH 501-2000 bounty
bounty between 501 and 2000 HUSH arm
something doesn't work on arm beginners
for new developers bug
may or may not be a bug build
problems building documentation
not enough information feature
new feature high priority
high priority i2p
related to i2p low priority
low priority medium priority
medium priority question
something is not clear release
release label or issue related to it testing
related to testing tor
related to tor wontfix
this won't be fixed

No Label 0-500 bounty 2001-5000 bounty 501-2000 bounty arm beginners bug build documentation feature high priority i2p low priority medium priority question release testing tor wontfix

Milestone

Set milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Assign users

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.