lnd icon indicating copy to clipboard operation
lnd copied to clipboard

neutrinorpc: add getblockhash

Open ErikEk opened this issue 3 years ago • 9 comments
trafficstars

The pr adds the ability to get a header hash of a block at a given block height from the neutrino sub-server. I would consider this a fix to #6688, this is how the bitcoin core api interface is designed (https://chainquery.com/bitcoin-cli/getblock).

ErikEk avatar May 07 '22 06:05 ErikEk

@bhandras Could you please review this.

ErikEk avatar Jun 30 '22 04:06 ErikEk

This is my first time reviewing an LND PR, but I can utACK it for now. If I get time i can test it soon(tm).

SachinMeier avatar Jul 16 '22 06:07 SachinMeier

@bhandras Thanks for taking the time to review it.

ErikEk avatar Jul 18 '22 20:07 ErikEk

@yyforyongyu If you have time; could you please review this.

ErikEk avatar Jul 18 '22 23:07 ErikEk

@yyforyongyu Thanks for review! I can look into adding itests. In that case I would add them to all the sub-server functions as a separate PR.

ErikEk avatar Jul 23 '22 17:07 ErikEk

@yyforyongyu Will add itests when this is merged.

ErikEk avatar Jul 27 '22 20:07 ErikEk

@yyforyongyu The method is accessible from both CLI and REST to answer your question. Just added access to them in the different commits which is a bit confusing.

ErikEk avatar Aug 02 '22 15:08 ErikEk

@erikek, remember to re-request review from reviewers when ready

lightninglabs-deploy avatar Aug 09 '22 16:08 lightninglabs-deploy

@guggero Maybe this can make it into 0.15.1?

ErikEk avatar Aug 10 '22 11:08 ErikEk