01:41:06 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 01:42:08 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 01:42:09 Nobody is looking at it 01:42:19 Yes, I know :) 01:42:22 Their new protocol looks like a beast. That's all I got so far 01:42:23 Just curious for the future 01:43:04 Id say: look at some other chain that implemented it (if any did) 01:43:28 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? 01:44:00 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 01:44:35 I initially proposed noise based on feedback from discussions at a monerokon then pivoted to ssl 01:45:46 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 01:46:46 We can probably roll our own for trezor if needed, it's not terribly difficult assuming they pick just one specific algorithm 01:46:58 It's just Noise_XX 01:48:25 Someone made a Bitcoin-only library for the protocol already, which apparently works and could perhaps be a useful reference: 01:48:29 https://github.com/coreyphillips/trezor-connect-rs 01:49:39 The noise protocol is just for safe 7 right? The others use the older existing protobuf scheme? 01:50:37 Yes, I believe so 01:51:32 I guess they did it because Bluetooth (and maybe) nfc, but damn is it annoying from our perspective 01:52:05 Actually nfc power limits should be too low for monero txes but maybe I'm craY 01:52:11 *crazy 04:36:02 maybe we should get the p2p SSL patch running on beta to get it some good testing 06:54:03 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. 12:17:10 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 12:18:41 TLS does, at least, save some headaches with not having to re-implement Noise for post-quantum... PQ noise extensions seem like a mess 12:20:45 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. 12:22:18 The patch was part of the original SSL patch PR and was removed by request, so easy to find. 12:22:58 Might be a pain to rebase after a decade of not being updated though. 12:23:56 I mean the reasoning behind dropping the P2P part seemed fair, but that was predicated on another solution being available. 16:04:52 .merge- 9473 16:04:52 Removed 16:16:34 Can anyone tell me why tx-notify isn't giving me the txid? 16:16:36 https://pasteview.com/xLa2xOv 16:17:15 i am seeing a GET request to: /monero/notification?tx_hash=/usr/bin/bash 16:23:36 What is in your monero-tx-notify script? 16:29:42 .. or try using a simple script that just redirects the txid to a file 16:41:54 script here: https://pasteview.com/ul13S8A 16:46:29 @moneromoooo the p2p SSL PR is basically that with some changes that were needed 16:55:22 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 17:02:33 it is executable. strange 19:29:09 Oh OK. Then it has a chance of going in then :D 20:04:07 Is this supposed to mean 'use' or 'unset'? 20:04:09 https://github.com/monero-project/monero/blob/master/src/device/device_ledger.cpp#L2319C20-L2319C24 20:04:47 Is this supposed to say 'use' or 'unset'? 21:31:44 Hello hello, I’m very interested in ring signatures. I’m happy to be here and looking to get involved and to contribute. 22:18:42 : I see an error in the log, not sure if its related 22:18:43 spawn src/common/spawn.cpp:114 Failed to execve: Exec format error 22:26:38 try a simple `echo $1 >> txids.txt` inside of the script