Skip to content

Add disable_outbound_lightning_connections flag - #1064

Open
tankyleo wants to merge 1 commit into
lightningdevkit:mainfrom
tankyleo:2026-08-disable-peer-networking
Open

tankyleo wants to merge 1 commit into
lightningdevkit:mainfrom
tankyleo:2026-08-disable-peer-networking

Conversation

@tankyleo

@tankyleo tankyleo commented Aug 20, 2026 •

Copy link
Copy Markdown
Contributor
    Add `disable_outbound_lightning_connections` flag

    Disable all outbound Lightning connections, including explicit API
    requests, with one check before DNS resolution and TCP/Tor connection
    creation. Inbound listening is controlled by listening_addresses; chain
    synchronization and other networking are unaffected.

    Co-Authored-By: HAL 9000

@ldk-reviews-bot

ldk-reviews-bot commented Aug 20, 2026 •

Copy link
Copy Markdown

👋 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.

@tankyleo
tankyleo requested a review from tnull August 20, 2026 13:43
@tankyleo

Copy link
Copy Markdown
Contributor Author

Curious your thoughts Elias, I understand this may be a hacky use case. See this issue for context: lightningdevkit/ldk-server#257

@tankyleo

Copy link
Copy Markdown
Contributor Author

If the node reconnects to the peer with a stale database, we get a panic, which I think is the correct course of action here; that's been the core motivation for this offline flag.

@tnull

tnull commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Curious your thoughts Elias, I understand this may be a hacky use case. See this issue for context: lightningdevkit/ldk-server#257

Hmm, I honestly would prefer to directly go for the full recovery mode we planned for a bit. Do you think this is time critical, or can we add it for v0.9?

@tankyleo

Copy link
Copy Markdown
Contributor Author

Curious your thoughts Elias, I understand this may be a hacky use case. See this issue for context: lightningdevkit/ldk-server#257

Hmm, I honestly would prefer to directly go for the full recovery mode we planned for a bit. Do you think this is time critical, or can we add it for v0.9?

Yes let's go for the full recovery mode (do you have an issue describing this?)

I think this can wait for 0.9, feel free to close this PR and we can re-open a separate proper recovery PR.

@tnull

tnull commented Aug 20, 2026 •

Copy link
Copy Markdown
Collaborator

Curious your thoughts Elias, I understand this may be a hacky use case. See this issue for context: lightningdevkit/ldk-server#257

Hmm, I honestly would prefer to directly go for the full recovery mode we planned for a bit. Do you think this is time critical, or can we add it for v0.9?

Yes let's go for the full recovery mode (do you have an issue describing this?)

I think this can wait for 0.9, feel free to close this PR and we can re-open a separate proper recovery PR.

No issue so far, but what we discussed was to have a special mode on builder or a separate utility that you'd give all the information you have (seed, list of known prior peers, etc) and it would try to claw all funds back that are possible, i.e., do force-close recovery but also retrieve possible_v2_counterparty_closed_balance_spks, and also do iterative full scans for the onchain wallet etc pp.

Actually, let me see if i can do a quick draft.

@tnull

tnull commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

Superseded by the recovery mode added in #1106

@tankyleo Does that address what you had in mind?

@tnull
tnull removed their request for review September 18, 2026 11:49
Disable all outbound Lightning connections, including explicit API
requests, with one check before DNS resolution and TCP/Tor connection
creation. Inbound listening is controlled by listening_addresses; chain
synchronization and other networking are unaffected.

Co-Authored-By: HAL 9000
@tankyleo
tankyleo force-pushed the 2026-08-disable-peer-networking branch from 18dff83 to adff576 Compare October 3, 2026 00:04
@tankyleo tankyleo changed the title Add disable_peer_networking flag Add disable_outbound_lightning_connections flag Oct 3, 2026
@tankyleo
tankyleo requested a review from tnull October 3, 2026 00:21
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.

3 participants