Skip to content

Log node URIs on startup using announcement addresses#135

Merged
tnull merged 2 commits intolightningdevkit:mainfrom
benthecarman:ann-adr
Feb 25, 2026
Merged

Log node URIs on startup using announcement addresses#135
tnull merged 2 commits intolightningdevkit:mainfrom
benthecarman:ann-adr

Conversation

@benthecarman
Copy link
Collaborator

Closes #130

Also adds them to the get info response

@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Feb 25, 2026

👋 Thanks for assigning @tnull 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.

@benthecarman benthecarman requested a review from tnull February 25, 2026 02:40
benthecarman and others added 2 commits February 25, 2026 12:44
Constructs connection URIs in the node_id@address format from the
announcement addresses, making it easy for clients to get ready-to-use
connection strings.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tnull tnull merged commit 6a42736 into lightningdevkit:main Feb 25, 2026
7 checks passed
@benthecarman benthecarman deleted the ann-adr branch February 25, 2026 19:39
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.

Connection string log should use announcement address, not listening address

3 participants