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

Add proof courier address field to Tap address #434

Closed
ffranr opened this issue Aug 8, 2023 · 2 comments
Closed

Add proof courier address field to Tap address #434

ffranr opened this issue Aug 8, 2023 · 2 comments
Assignees

Comments

@ffranr
Copy link
Contributor

ffranr commented Aug 8, 2023

A Tap address should include a mandetory proof courier address.

The receiving node during the asset send procedure could use this new field to contact a proof courier service and retrieve proof(s) associated with then send event.

In other words: In order to receive and asset, the receiver generates a tap address. That tap address will contain a proof courier address. The sender submits the transfer proof to that proof courier. Which the receiver can then retrieve.

This new field should support multiple proof courier protocols:

universerpc://the.host.name:port
hashmail://mailbox.server:port

This is a prerequisite to #344

@ffranr
Copy link
Contributor Author

ffranr commented Aug 8, 2023

PR for adding TLV field: #450

@ffranr ffranr added v0.3 and removed needs triage labels Aug 8, 2023
@guggero guggero linked a pull request Aug 9, 2023 that will close this issue
@ffranr ffranr changed the title Add optional proof courier address to Tap address Add mandatory proof courier address to Tap address Aug 9, 2023
@ffranr ffranr changed the title Add mandatory proof courier address to Tap address Add proof courier address field to Tap address Aug 9, 2023
@ffranr ffranr moved this from 👀 In review to 🏗 In progress in Taproot-Assets Project Board Aug 9, 2023
@ffranr ffranr moved this from 🏗 In progress to ✅ Done in Taproot-Assets Project Board Aug 22, 2023
@guggero
Copy link
Member

guggero commented Aug 22, 2023

Fixed by #450.

@guggero guggero closed this as completed Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
2 participants