Skip to content

Refactor/silentpayments transaction shared secret - #141

Open
Sosthene00 wants to merge 7 commits into
masterfrom
refactor/silentpayments-transaction-shared-secret
Open

Refactor/silentpayments transaction shared secret#141
Sosthene00 wants to merge 7 commits into
masterfrom
refactor/silentpayments-transaction-shared-secret

Conversation

@Sosthene00

Copy link
Copy Markdown
Collaborator
  • Replace SharedSecret with TransactionSharedSecret, that carries both the ECDH point and recipient scan key.
  • Introduce PublicTweakData for receiver-side shared secret derivation using TransactionInputs.
  • Extract hash helpers (calculate_label_hash, calculate_shared_secret_hash) and add NormalizedSecretKey type for taproot key normalization.
  • Keep the existing PartialSecret / calculate_partial_secret sending API unchanged for now.

@Sosthene00
Sosthene00 force-pushed the refactor/silentpayments-transaction-shared-secret branch from 56397f2 to 98cf407 Compare June 23, 2026 13:20
@Sosthene00
Sosthene00 force-pushed the refactor/silentpayments-transaction-shared-secret branch from 98cf407 to c2b7fa8 Compare July 22, 2026 11:12
@Sosthene00
Sosthene00 force-pushed the refactor/silentpayments-transaction-shared-secret branch 3 times, most recently from fb6d7c6 to d828248 Compare August 12, 2026 10:35
@Sosthene00
Sosthene00 force-pushed the refactor/silentpayments-transaction-shared-secret branch 2 times, most recently from a392e7c to 76d89e6 Compare August 26, 2026 11:38
Only accepts selections whose implied fee rate stays within the
requested rate (modulo vbyte rounding): selections that would absorb
the leftover excess into the fee are rejected, and among the accepted
ones the fee is minimized. When no selection can honor the rate the
strategy is simply absent from the results, leaving callers free to
fall back to the other strategies.
@Sosthene00
Sosthene00 force-pushed the refactor/silentpayments-transaction-shared-secret branch from 76d89e6 to a721c7d Compare September 8, 2026 08:43
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