Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BOLT 4: Add sender authentication #946

Closed
wants to merge 1 commit into from

Conversation

joostjager
Copy link
Collaborator

Adds the possibility for the sender to optionally authenticate themselves. More background: https://lists.linuxfoundation.org/pipermail/lightning-dev/2021-December/003422.html

This may not be the most popular PR, but sender authentication is possible using custom records already and perhaps it is better to spec it out here rather than letting it emerge organically. Otherwise we may get the same situation that we currently have with payment types. LNURL is ahead with sender auth already too.

Also we have the opportunity to set a standard for sender authentication that preserves privacy better than adding a signature.

@t-bast
Copy link
Collaborator

t-bast commented Dec 20, 2021

IMHO there will be many different ways we could want to do "some kind of sender authentication", probably too many have them in the BOLTs.

BLIPs feel like a better candidate for this, at least for now?

@BitcoinErrorLog
Copy link

This probably shouldn't be part of the spec. Even LNURLauth doesn't need to be specific to LN or a LN node. Why would each node need to auth individually anyway? What if I own two nodes? Etc, etc, etc.

@TheBlueMatt
Copy link
Collaborator

LNURL is ahead with sender auth already too.

I assume (or, well, maybe hope?) that most wallets haven't implemented that? LNURL has many parts and only some are widely implemented.

At a high level, I agree with @t-bast here - one major issue is what, exactly, are you authenticating? Chat protocols shouldn't be explicitly authenticating, probably, but doing some kind of double-ratchet handshake to establish a secure channel instead. KYC protocols probably want an explicit authentication of a node_id, but only a pre-registered one. Other protocols may want a random one-off one, etc.

@joostjager
Copy link
Collaborator Author

Will convert this into a BLIP, which is indeed more suitable.

@vincenzopalazzo
Copy link
Contributor

Concept ack on the idea to have this definition in the BLIP.

@joostjager
Copy link
Collaborator Author

Created bLIP, closing this PR.

lightning/blips#14

@joostjager joostjager closed this Feb 4, 2022
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.

5 participants