Oli

Results 502 comments of Oli
trafficstars

@arshbot any thoughts on the discussion above? Should we try to pick this up again for 0.15? I'm going to remove my request for review until we know how we...

Thanks for the PR. I took a quick look at the keyless signing with `cosign`. From what I understand this doesn't really improve the security of the generated images. Sure,...

Option 1 sounds like adding a whole new "filter language parsing" module to `lnd`... Though maybe there are libraries out there. Perhaps at some point (after migrating everything to "native"...

Since you last rebased we made the `tor` package its own submodule. That fixed a dependency issue but makes it a bit harder to create PRs that change the functionality,...

Hmm, this is a tough one. I guess what I forgot to mention is that the `tor` submodule, to be independent and to work standalone and to avoid circular dependencies,...

Sounds good! Will review once you give me the heads up.

I think the `OnionFile` struct (and its file) can be moved directly to the main `tor` package/submodule. Or it can stay in the `tor/onionfile` package (but shouldn't be a submodule,...

Can be rebased now and use the `tor/v1.1.0` dependency in `go.mod`.

Oops, another merge requires this PR to be rebased.

Updated to merged `btcwallet` dependency. This is ready for full review.