Skip to content

Add network field to GetNodeInfoResponse - #198

Merged
benthecarman merged 2 commits into
lightningdevkit:mainfrom
randomlogin:add-network-to-node-info
May 6, 2026
Merged

Add network field to GetNodeInfoResponse#198
benthecarman merged 2 commits into
lightningdevkit:mainfrom
randomlogin:add-network-to-node-info

Conversation

@randomlogin

Copy link
Copy Markdown
Contributor

Expose the Bitcoin network the node is running on (e.g., "bitcoin", "testnet", "signet", "regtest") via the get-node-info API.

This was merged in lightningdevkit/ldk-node#892

I've bumped the dependency revision of ldk-node to the latest one before breaking changes are introduced (lightningdevkit/ldk-node#850).

@ldk-reviews-bot

ldk-reviews-bot commented May 5, 2026

Copy link
Copy Markdown

👋 Thanks for assigning @benthecarman as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

Comment threadldk-server-grpc/src/proto/api.proto Outdated
@randomlogin
randomloginforce-pushed the add-network-to-node-info branch from 928d92a to 20b5d8dCompareMay 5, 2026 15:04
@benthecarman
benthecarman removed the request for review from valentinewallaceMay 5, 2026 15:08
Comment threadldk-server/src/util/proto_adapter.rs Outdated
Comment threadldk-server/Cargo.toml
@randomlogin
randomloginforce-pushed the add-network-to-node-info branch from 20b5d8d to 63e7184CompareMay 5, 2026 18:15
@benthecarman
benthecarman merged commit bf4d8bd into lightningdevkit:mainMay 6, 2026
10 checks passed
Sign up for freeto 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.

4 participants

@randomlogin@ldk-reviews-bot@benthecarman@Anyitechs