Yong

Results 102 comments of Yong

> Mind elaborating? Do you mean in general or specifically in this PR? Meant in this PR. Since we are moving the code outside of `server.go`, it's finally a good...

Is this ready for another round?

Turns out the neutrino flakes had already been found and fixed here😂 @guggero

What's the status here? Still ongoing? cc @tvolk131

I did find [something](https://github.com/lightningnetwork/lnd/pull/6683/commits/782e0bc91d8a4b8efddf6028fb40eb8e01f67484#diff-040acca58de99153a29ab6ac4871acd49fc5e1af78cc7dc6e75c3e78bd065254R366), not sure if it's related. > Is it okay for us to work on it? I'm asking in case someone is already looking into this. @lsunsi...

@lsunsi Thanks for the detailed explanation! Yeah in this case I'd say please go ahead and make a PR!

> In this case, after I look up the transaction it isn't present, which confirms the suspicion that we are properly removing the transaction as expected if policy is violated....

> we have already sent our tx out into the wild and cant ever be 100% sure that it has not actually broadcast. I don't think this is something we...

@Crypt-iQ If we want to handle more than tx-size error, it'd be nice but another story. This `testmempoolaccept` is dope and can be implemented in `btcwallet`. I think our abstractions...

Closed as the reported issue has been fixed.