Skip to content
This repository has been archived by the owner on Feb 3, 2025. It is now read-only.

DLCs #888

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

DLCs #888

wants to merge 2 commits into from

Conversation

benthecarman
Copy link
Collaborator

@benthecarman benthecarman commented Dec 1, 2023

Opening for now so I can better document changes I need to make

Currently can open and close DLCs with my dummy function test_send_dlc_offer.

Has some current problems

  • Privacy leak same keys used across all DLCs (need changes to rust_dlc)
  • Same nostr key for DLCs Cannot statically receive bets without this
  • Need to do proper coin selection, not just use them all
  • Currently using damus relay with random kind, need to do this properly
  • Encrypt raw bytes of DLC messages instead of base64 encodings (needs rust-nostr v26)
  • Respond with errors
  • Docs
  • Tests

Some other todos for usability

  • Make functions for decoding oracle announcements and attestations
  • Better wasm model

Longer term todos

  • nostr market place
  • local social tagging
  • dlc "zaps"

@TonyGiorgio
Copy link
Contributor

Code looks pretty clean and self contained, nice job so far. What's the status on it?

@TonyGiorgio
Copy link
Contributor

needs rebase

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants