Skip to content
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

Add public request method and getmempoolinfo to the api #1

Merged
merged 2 commits into from
Jul 13, 2022

Conversation

nepet
Copy link
Contributor

@nepet nepet commented Jul 13, 2022

The RPC api is only a subset of the bitcoind RPC api. This Pr adds a public request method that can use an arbitrary jrpc2.Method to pass to the bitcoin client.

This Pr also adds getmempoolinfo to the api.

nepet added 2 commits July 12, 2022 18:39
…api.

Not all bitcoind RPC commands are reflected by glightning. This way
we can bypass the api to send raw RPC commands to bitcoind such that
we do not have to file an invoice every time we need an api call
that is missing.
@nepet nepet requested a review from wtogami July 13, 2022 13:44
@nepet
Copy link
Contributor Author

nepet commented Jul 13, 2022

@wtogami we need this pr for the fee estimation rework on ElementsProject/peerswap

nepet added a commit to nepet/peerswap that referenced this pull request Jul 13, 2022
This is a temporary fix so that we can test the rework of the fee
estimation. This is meant to be removed before we merge to master.

Check out ElementsProject/glightning#1
@wtogami
Copy link
Contributor

wtogami commented Jul 13, 2022

utACK

@wtogami wtogami merged commit 49a9a8e into ElementsProject:master Jul 13, 2022
nepet added a commit to nepet/peerswap that referenced this pull request Jul 17, 2022
This is a temporary fix so that we can test the rework of the fee
estimation. This is meant to be removed before we merge to master.

Check out ElementsProject/glightning#1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants