Skip to content

Add network field to GetNodeInfoResponse#198

Open
randomlogin wants to merge 2 commits intolightningdevkit:mainfrom
randomlogin:add-network-to-node-info
Open

Add network field to GetNodeInfoResponse#198
randomlogin wants to merge 2 commits intolightningdevkit:mainfrom
randomlogin:add-network-to-node-info

Conversation

@randomlogin
Copy link
Copy Markdown

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
Copy link
Copy Markdown

ldk-reviews-bot commented May 5, 2026

👋 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 thread ldk-server-grpc/src/proto/api.proto Outdated
@randomlogin randomlogin force-pushed the add-network-to-node-info branch from 928d92a to 20b5d8d Compare May 5, 2026 15:04
@randomlogin randomlogin requested a review from benthecarman May 5, 2026 15:07
@benthecarman benthecarman removed the request for review from valentinewallace May 5, 2026 15:08
Comment thread ldk-server/src/util/proto_adapter.rs Outdated
Comment thread ldk-server/Cargo.toml
@randomlogin randomlogin force-pushed the add-network-to-node-info branch from 20b5d8d to 63e7184 Compare May 5, 2026 18:15
@randomlogin randomlogin requested a review from benthecarman May 5, 2026 18:36
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.

4 participants