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

Setup onion error packet #354

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

doitian
Copy link
Member

@doitian doitian commented Nov 29, 2024

Setup the framework to return payment error via onion error packet.

The spec can be found here: https://github.com/cryptape/fiber-sphinx/blob/develop/docs/spec.md#returning-errors

The encryption has not been enabled yet so I can split the changes into several PRs. The encryption can be enabled later without breaking the network protocol because the decryption phrase will check whether the error has been encrypted or not.

Refs #198

@doitian doitian force-pushed the feature/onion-returning-errors branch 3 times, most recently from 905f7c5 to 07192c9 Compare November 29, 2024 04:43
@doitian doitian mentioned this pull request Nov 29, 2024
@doitian doitian changed the title Setup onion error packet [Rebasing] Setup onion error packet Dec 3, 2024
Setup the framework to return payment error via onion error packet.

The spec can be found here: https://github.com/cryptape/fiber-sphinx/blob/develop/docs/spec.md#returning-errors

The encryption has not been enabled yet so I can split the changes into
several PRs. The encryption can be enabled later without breaking the
network protocol because the decryption phrase will check whether the
error has been encrypted or not.
@doitian doitian force-pushed the feature/onion-returning-errors branch from 07192c9 to 7b5cd90 Compare December 3, 2024 01:38
@doitian doitian changed the title [Rebasing] Setup onion error packet Setup onion error packet Dec 3, 2024
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