-
bonedaddy
if you have a reserve proof, but dont know the balance that was proved with it, how can you determine the balance? i checked the monero-wallet-cli, and monero-wallet-rpc calls, and it doesnt appear possible, as `check_reserve_proof` simply checks to see if the proof is valid iirc
-
moneromooo
IIRC the balance proof proves ownership of outputs. So you can sum up all those outputs, check they're not spent.
-
moneromooo
I'm not sure how it's laid out internally. I assume you get the amount masks.
-
bonedaddy
ah i see, i will try that, thanks.
-
UkoeHB_
bonedaddy: I believe `check_reserve_proof()` generates the amount stored in the outputs; the proof itself has all the information you need to get the amounts
-
bonedaddy
UkoeHB: how to parse the proof though, i cant seem to find any docs explaining how the proof is structured
-
UkoeHB_
in ztm2 section 8.1.6 I discussed reserveproofs, hopefully that will point you in the right direction
-
bonedaddy
thanks ill check that
-
moneromooo
struct reserve_proof_entry (wallet2.h)
-
bonedaddy
nice, thanks
-
UkoeHB_
looks like the return-by-reference variables `total` and `spent` from `check_reserve_proof` refer to the unspent amount `unspent amount = total - spent`
-
UkoeHB_
moneromooo: the struct doesn't have any amounts
-
moneromooo
Looks like it decodes it with the shared secret.
-
UkoeHB_
-
UkoeHB_
yeah idk how the rpc works at all, but it looks like `check_reserve_proof` is the command you want, and the response should be response_t from that struct I linked bonedaddy
-
bonedaddy
awesome thanks for finding this! i think the docs for the wallet-rpc are slightly incorrect then.
getmonero.org/resources/developer-g…wallet-rpc.html#check_reserve_proof the `outputs` section for the check_reserve_proof command doesnt include the `total` field.
-
UkoeHB_
If you scroll down a few lines the demo output of the command shows total and spent fields
-
moneromooo
.merge+ 6840
-
xmr-pr
Added
-
woodser_
updating my webassembly project to v0.17.0.0, it appears crypto from the supercop submodule is now being used. I could link to the new source files, but I notice it'll fall back to `using ::crypto::generate_key_derivation;` if monero_crypto_generate_key_derivation isn't defined in src/crypto/wallet/crypto.h. Wondering if it's better to link to the supercop submodule source files moving forward, or if it's sufficient to u
-
woodser_
se the fall back and if there's a flag to do that easily?
-
moneromooo
Well, it's assembler. Can webassembly even use raw asm ?
-
selsta
.merge 6835 6834
-
selsta
.merge+ 6835 6834
-
xmr-pr
Added
-
Alex_LocalMonero
@moneromooo, hi. Since the release of v0.16 with dandelion we've been having issues with transactions sometimes not propagating and being stuck in pending on the local wallet. We have to manually clear mempool and rescan spent each time. Is this is a known issue and how do we fix it? This has never happened before v0.16
-
moneromooo
Might be the node(s) you relayed to being assholes and dropping them.
-
moneromooo
relay_tx TXID *should* work, but it will likely broadcast to all.
-
moneromooo
vtnerd_ might have a better answer.
-
selsta
would deleting p2pstate.bin theoretically maybe help with malicious peers? so that you get completely fresh peers?
-
Alex_LocalMonero
@moneromooo, what would be the command to manually add a trusted node to always broadcast to?
-
selsta
--add-priority-peer or so
-
moneromooo
You might get back to them.
-
Alex_LocalMonero
@selsta, when adding a peer should we use the same port as the RPC wallet connects to or is there a different port usually?
-
selsta
18080 is default
-
selsta
p2p port
-
selsta
I think
-
moneromooo
It is.
-
Alex_LocalMonero
Thanks.
-
Alex_LocalMonero
@selsta, no --add-priority-peer, but there is add-priority-node. So, I assume the config line would be add-priority-node=node.trusted.com:18080?
-
Alex_LocalMonero
Is there a command in a running daemon that we could use to verify that our node is now connected to the trusted node?
-
dEBRUYNE
print_pl will show the peerlist
-
dEBRUYNE
That should allow you to verify that you are connected to the node that was specified upon startup
-
dsc_
or `sudo ngrep -qt -W byline 'dst host 8.8.8.8'` to view traffic flying by, which is usually a good indication :p
-
dsc_
(replace IP)
-
Alex_LocalMonero
Great, thanks.
-
moneromooo
print_cn
-
moneromooo
print_pl will show all peers
-
moneromooo
Known peers, rather than the ones you're connected to.
-
grydz
selsta, version 1.7.1 of Ledger's Monero app which fix transfers on testnet should be available tomorrow.
-
selsta
awesome
-
grydz
I'll let you know on GitHub when it's available :)
-
sarang
:)
-
sarang
Thanks grydz!
-
grydz
Thanks to you guys for your help ;)
-
selsta
.merge+ 6841 6842
-
xmr-pr
Added
-
selsta
.merges
-
xmr-pr
6834 6835 6840 6841 6842
-
iamthesenate
Hello :) Does anybody know of a working stagenet faucet right now? I need to do some tests using stagenet moneroj...
-
selsta
-
iamthesenate
This one did not work for me, sadly :( I did not get the 10 moneroj and also started getting the 'You are blocked' message after...
-
vtnerd_
Alex_LocalMonero: a rescan on the wallet side wouldn't fix the issue? The mempool had to be cleared also? Or .. ?
-
Alex_LocalMonero
@vtnerd_, if you don't flush the mempool there's a risk of the tx being eventually broadcast as the tx is still in the mempool even after rescan_spent on the wallet.
-
Alex_LocalMonero
@moneromooo @vtnerd_ , I'd like to reiterate that this has never happened before v0.16. After v0.16 it became a semi-regular occurrence.
-
moneromooo
Before 0.16, transactions were broadcasted to all your peers.
-
moneromooo
Last I looked, 50% of my 8 out peers were lying nodes.
-
Inge-
lying?
-
moneromooo
Deliberately untruthful.
-
Alex_LocalMonero
@moneromooo, why would nodes be deliberately untruthful?
-
Alex_LocalMonero
To DoS the network?
-
moneromooo
Because their human operator forces them to.
-
Alex_LocalMonero
@moneromooo, apart from manually adding trusted nodes, is there a way to strengthen the protocol to avoid this problem without having to rely on people knowing to manually add trusted nodes?
-
moneromooo
Feel free to suggest ideas.
-
Alex_LocalMonero
In order to do that I'd need to better understand how a node lies. Does it not broadcast the tx further after receiving it?
-
sarang
Haven't read back... are you referring to black hole attacks on Dandelion routing?
-
Alex_LocalMonero
Oh, so this is a known issue then.
-
Alex_LocalMonero
Yeah sarang, we've had issues since v0.16 that sometimes tx don't get propagated.
-
sarang
At all?
-
Alex_LocalMonero
At all.
-
moneromooo
It doesn't matter. You can assume it'll do the worst you can imagine.
-
sarang
There's a Dandelion embargo period against black-hole attacks, which is what I was referring to
-
selsta
I have only had transactions taking 2-3 minutes until they show up in mempool, but I don’t send as many tx as localmonero.
-
moneromooo
At all is not right, your own node should timeout after a feww minutes and broadcast.
-
selsta
Timeout is only 10 seconds IIRC.
-
moneromooo
Seems... small.
-
selsta
Then I misremember :)
-
sarang
The embargo timer does have some randomness as well
-
Alex_LocalMonero
Yeah, about 0.2% of our outgoing transactions don't propagate. We have to flush tx pool and rescan spent.
-
Alex_LocalMonero
How's this for a mitigation: have your node rebroadcast choosing different peers if you don't get the tx from another node after a certain time.
-
sarang
If the embargo expires, the node goes into fluff mode
-
sarang
and broadcasts to all peers
-
Alex_LocalMonero
How long is the embargo?
-
selsta
-
Alex_LocalMonero
That's weird, we've had transactions stuck on pending for multiple days.
-
selsta
that would explain the issue with tx taking 2-3 minutes to show up in mempool in some cases
-
selsta
maybe increasing out_peers helps so that it gets broadcasted to more nodes
-
selsta
CipherTrace CEO was talking about methods to circumvent D++ specifically, so it is likely that they spam the network with nodes that black hole tx.
-
moneromooo
Oh, so they switched from passive spying to actual sabotage ?
-
moneromooo
173 is right about what I remembered.
-
sarang
They admitted to injecting transactions