[Feature] Add ability for users to set a default change address #234
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fluxpools have reported that they would like the ability to set a default change address for when the are sending payments to miners. The reason for this feature request is because normal transactions generate a new address to be the change address causing the wallet to grow in size.
To allow this type of feature fluxd now allows two new parameters to be passed to fluxd when starting the application or they can be placed in the flux.conf.
The default change address will be assigned if it was passed to the application in either of the mechanisms above, it is a valid address for the network is belongs to, and it will only work with the following rpc calls
The rpccall sendfrom defaults to sending the change back to the address that it is sending from. So the default change address will not be used when this rpc is used.
Tests to be completed before merging: