-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PENDING]neutrinorpc: remove getblockhash #7699
base: master
Are you sure you want to change the base?
[PENDING]neutrinorpc: remove getblockhash #7699
Conversation
ea9bb8e
to
e95f05d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! Not sure if we could remove an RPC like this. Maybe we need to first deprecate it in one release and then remove it in the next following release?
@yyforyongyu Ok, I will write a pr to deprecate it for 0.17 and remove it for 0.18 or 0.19 then. Just assumed we could get rid of it because it hasnt been in the code base for very long. |
@yyforyongyu Or actually... the reason I removed it was because it has already been removed from the lncli interface (https://github.com/lightningnetwork/lnd/blob/2fd40d16f1ec7a93d17870d27dfc3d488d34a7c9/cmd/lncli/neutrino_active.go#LL238C5-L243C23). Still want to me to deprecate it? |
e95f05d
to
337535b
Compare
337535b
to
c0a5397
Compare
Yeah, if a mobile app developer already started using this after updating to 0.16, it would be a bit premature to remove it again in 0.17, even if there's an alternative. |
@guggero @yyforyongyu Deprecated neutrinorpc getblockhash in #7712. |
Removes the getblockhash function from the neutrino sub-server. The functionality and endpoint has been moved to the chainrpc sub-server in 0.16.
Waiting for lnd 0.18 or 0.19...