2018-08-15 05:35:22
[snowblossomcoin/snowblossom] Issue closed by fireduck64
GitHub
2018-08-15 05:38:27
@Eric has joined the channel
Eric
2018-08-15 05:43:49
We need to do this anyways. The TransactionUtil.makeTransaction will a bunch of parameters needs improvement.
So the new one will have a configuration object that specifies all the parameters.
Options to support:
• Broadcast or just return tx data
• Sign or not
• Split change if larger than X
• Change (explicit addresses, random address from wallet, fresh address (default))
• Inputs (explicit list of candidates, confirmed then pending (default), confirmed only)
• Fee (explicit, based on node recommendation)
Should be supported in java library and via json rpc
GitHub
2018-08-15 05:44:43
Need facility to sign a transaction. This allows multisig use cases as well, since the partially signed transaction can be handed around.
GitHub
2018-08-15 05:47:10
probably json rpc callback. Register with node?
GitHub
2018-08-15 05:48:19
make sure the access functions are accessible via jsonrpc.
GitHub
2018-08-15 06:02:54
@Eric can you give me an idea of your normal flow? Transaction wise, snowblossom is just like bitcoin.
Fireduck
2018-08-15 15:53:12
@bitbumper has joined the channel
bitbumper
2018-08-15 15:54:20
Eric's gotta run but I can take the question, hope it's cool that I invited myself. I write our coin relay software.
bitbumper
2018-08-15 15:55:14
Not sure what you mean by normal flow? Our relays are setup as a set of tasks that get run periodically. We have tasks like ScanDespoits, ConfirmDeposits, etc.
bitbumper
2018-08-15 15:57:30
Do you sweep deposits into a pool?
Fireduck
2018-08-15 15:58:26
Not as a matter of routine, we just construct our withdraw TXs from available unspents.
bitbumper
2018-08-15 15:59:19
Bitcoin core can get slow with 1000+ UTXOs, so we occasionally run sweeps as a maintenance task.
bitbumper
2018-08-15 16:09:07
Got it
Fireduck
2018-08-15 16:09:52
So you need to make sure the change from that tx doesn't hit a deposit address.
Fireduck
2018-08-15 16:11:07
Yeah, we have a concept of "internal" addresses that can be cold storage, change, etc. Deposits to these addresses are monitored but do not credit user accounts.
bitbumper
2018-08-15 16:12:26
Makes sense.
Fireduck
2018-08-15 16:12:49
Can you take a look at issue 108?
Fireduck
2018-08-15 16:13:22
It is notes for new transaction create library
Fireduck
2018-08-15 16:13:58
I want to make sure it covers your needs
Fireduck
2018-08-15 16:14:36
Sure, let me take a look
bitbumper
2018-08-15 16:19:43
Yeah looks like it would have everything we need. A few thoughts:
• If we let the node pick the fee, it would be nice to have the fee information in the return payload for logging/analytics
• We would likely be giving an explicit list of UTXO candidates and asking for an unsigned transaction
• Split change is a nice feature, although not strictly needed.
bitbumper
2018-08-15 16:20:29
Cool
Fireduck