nice, great to see more NWC implementations. is there anything that you dislike on our LND implementation?

GitHub

GitHub - getAlby/nostr-wallet-connect: Nostr Wallet Connect (NIP-47) application to allow apps to connect to your node

Nostr Wallet Connect (NIP-47) application to allow apps to connect to your node - getAlby/nostr-wallet-connect

it supports the LND backend (which is also used for Umbrel for example)

Rather use something I can audit myself

you mean because of programming language? or what should be changed?

I just like using software I wrote myself.

I zap

🔥

Does this work with one tap zaps?

Changing Allow self payment: true 💕 Code looks great will test this out when back from Miami

Nice! 👏

ありがとう❗️🎉

@benthecarman's implementation looks more minimal with less LoC overall. Plus rust > go when it comes to safety. I've been using alby's nwc and it crashes sometimes when the connection to a relay drops. Also, the support for connecting to a custodial alby account makes me uncomfortable since now I know there is code that can talk to alby's servers... I will try this one and compare them.

I'm using the clone I did about 2 weeks ago. My workaround was to add RestartOnFailure to the systemd service I created for it. It still bothers me that it can crash periodically but it was already fixed, have to check. I use the lnd backend and it's nice that I can audit the code but I would still prefer a more streamlined software that does not have that feature (connecting to 3rd party custodial wallets). This is really a minor nitpick, I'm just splitting hairs at this point and bitching about software I use and enjoy and did not contribute 😂

Yeah I just like mine because it's stupid simple, 300 lines of code, a couple config flags, and that's it. Adding new rpc commands should be simple but for now will handle it just fine.

Why would you want that? You're just gonna do re-balances for no reason

To fake zaps on my notes of course :) Was just making a bad dev joke, I'll try harder next time