-
m-relay
<jpk68:matrix.org> As it seems the new Trezor wallets rely on Noise for their wire protocol, I was wondering what the preferred approach might be to implementing such a protocol in core. Regarding Noise, specifically
-
m-relay
<jpk68:matrix.org> Neither libsodium or OpenSSL support Noise (just its primitives), so it would have to use an existing Noise library (of which none are very widely adopted), or we could roll our own, I suppose
-
m-relay
<ofrnxmr:xmr.mx> Nobody is looking at it
-
m-relay
<jpk68:matrix.org> Yes, I know :)
-
m-relay
<vtnerd:monero.social> Their new protocol looks like a beast. That's all I got so far
-
m-relay
<jpk68:matrix.org> Just curious for the future
-
m-relay
<ofrnxmr:xmr.mx> Id say: look at some other chain that implemented it (if any did)
-
m-relay
<jpk68:matrix.org> Haha, I came to the same conclusion. I did read your writeup on the proposed P2P encryption support... did you have a Noise implementation in mind to use for that?
-
m-relay
<jpk68:matrix.org> There is also the option of using Trezor's Rust library if/when it comes out, or making some sort of wrapper for their Python code
-
m-relay
<vtnerd:monero.social> I initially proposed noise based on feedback from discussions at a monerokon then pivoted to ssl
-
m-relay
<vtnerd:monero.social> There were some interesting features, but it didn't seem worth the effort. In the end the ssl patch has received little reviews anyway, so perhaps I should've just gone for the noise implementation
-
m-relay
<vtnerd:monero.social> We can probably roll our own for trezor if needed, it's not terribly difficult assuming they pick just one specific algorithm
-
m-relay
<jpk68:matrix.org> It's just Noise_XX
-
m-relay
<jpk68:matrix.org> Someone made a Bitcoin-only library for the protocol already, which apparently works and could perhaps be a useful reference:
-
m-relay
-
m-relay
<vtnerd:monero.social> The noise protocol is just for safe 7 right? The others use the older existing protobuf scheme?
-
m-relay
<jpk68:matrix.org> Yes, I believe so
-
m-relay
<vtnerd:monero.social> I guess they did it because Bluetooth (and maybe) nfc, but damn is it annoying from our perspective
-
m-relay
<vtnerd:monero.social> Actually nfc power limits should be too low for monero txes but maybe I'm craY
-
m-relay
<vtnerd:monero.social> *crazy
-
m-relay
<jberman:monero.social> maybe we should get the p2p SSL patch running on beta to get it some good testing
-
tobtoht
I'm very much not a fan of adding a large number of rust crates or re-introducing Python to our supply chain to support one vendor's custom wire protocol.
-
m-relay
<jpk68:matrix.org> It is worth noting that the proposed Noise path for P2P encryption is very different than the variant of Noise used by Trezor. Implementing both could be a pain and it might be worth adopting a library like `noise-c` for that
-
m-relay
<jpk68:matrix.org> TLS does, at least, save some headaches with not having to re-implement Noise for post-quantum... PQ noise extensions seem like a mess
-
moneromooo
It is beyond ridiculous, and starting to look suspicious now. There was a P2P SSL patch many years ago, and a different impl has been around the corner for years. Someone who is not me should just find the old patch and rebase it.
-
moneromooo
The patch was part of the original SSL patch PR and was removed by request, so easy to find.
-
moneromooo
Might be a pain to rebase after a decade of not being updated though.
-
moneromooo
I mean the reasoning behind dropping the P2P part seemed fair, but that was predicated on another solution being available.
-
selsta
.merge- 9473
-
xmr-pr
Removed
-
Guest4708
Can anyone tell me why tx-notify isn't giving me the txid?
-
Guest4708
-
Guest4708
i am seeing a GET request to: /monero/notification?tx_hash=/usr/bin/bash
-
m-relay
<ofrnxmr:monero.social> What is in your monero-tx-notify script?
-
m-relay
<ofrnxmr:xmr.mx> .. or try using a simple script that just redirects the txid to a file
-
Guest4708
-
m-relay
<jberman:monero.social> @moneromoooo the p2p SSL PR is basically that with some changes that were needed
-
m-relay
<ofrnxmr:xmr.mx> guest4708, i just tested with my own script of simple `echo $1 >> txids.txt` and it worked. Make sure that your monero-tx-notify is executable
-
Guest4708
it is executable. strange
-
moneromooo
Oh OK. Then it has a chance of going in then :D
-
m-relay
<jpk68:matrix.org> Is this supposed to mean 'use' or 'unset'?
-
m-relay
-
m-relay
<jpk68:matrix.org> Is this supposed to say 'use' or 'unset'?
-
m-relay
<alphonse86:matrix.org> Hello hello, I’m very interested in ring signatures. I’m happy to be here and looking to get involved and to contribute.
-
Guest4708
<m-relay>: I see an error in the log, not sure if its related
-
Guest4708
spawn src/common/spawn.cpp:114 Failed to execve: Exec format error
-
m-relay
<ofrnxmr:xmr.mx> try a simple `echo $1 >> txids.txt` inside of the script