Skip to content

Use patched LNbits image and bypass self-onion zaps#355

Merged
macterra merged 2 commits intomainfrom
chore/use-patched-lnbits-image
Apr 2, 2026
Merged

Use patched LNbits image and bypass self-onion zaps#355
macterra merged 2 commits intomainfrom
chore/use-patched-lnbits-image

Conversation

@macterra
Copy link
Copy Markdown
Collaborator

@macterra macterra commented Apr 2, 2026

What changed

  • switch the lightning compose stack to the published patched LNbits image ghcr.io/archetech/lnbits:1.5.3-archetech.1
  • bypass Tor self-loopback for local DID-to-DID zaps when the published Lightning endpoint resolves back to this same stack
  • record the repo workflow note to prefer gh for GitHub write operations

Why

  • we wanted to use the Archetech-patched LNbits image instead of the local build
  • local zaps on localhost were failing because lightning-mediator tried to reach this stack's own onion callback through the Tor SOCKS proxy and timed out
  • local recipients can be served directly over the internal Docker network without changing remote onion behavior

Impact

  • the lightning stack now pulls the patched LNbits image directly
  • localhost-to-localhost zaps succeed again for recipients published on the same stack
  • remote onion recipients still use the Tor proxy path

Validation

  • pulled and restarted lnbits on ghcr.io/archetech/lnbits:1.5.3-archetech.1
  • reproduced the original timeout from inside lightning-mediator
  • rebuilt and restarted lightning-mediator
  • verified a local zap succeeds after the self-onion bypass change
  • verified lightning-mediator health is green after restart

@macterra macterra marked this pull request as ready for review April 2, 2026 00:30
@macterra macterra merged commit b094573 into main Apr 2, 2026
16 checks passed
@macterra macterra deleted the chore/use-patched-lnbits-image branch April 2, 2026 00:37
@macterra macterra mentioned this pull request Apr 2, 2026
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.

1 participant