-
Notifications
You must be signed in to change notification settings - Fork 575
feat: pay push --confidential / confidential mpp charges #381
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
base: main
Are you sure you want to change the base?
Changes from all commits
4149783
60d6ec8
4fb03b5
1adbd3e
564ba6e
bc2f3cf
3330a99
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -103,15 +103,16 @@ qrcode = { version = "0.14", default-features = false } | |
| mime_guess = { version = "2.0", default-features = false, features = ["rev-mappings"] } | ||
|
|
||
| # Solana MPP (includes keypair loading + charge + subscription + authenticate | ||
| # builders). Tracks pay-kit `main`; PR #154 merged the SIWMPP work so the | ||
| # stable branch is the source of truth again. | ||
| solana-mpp = { git = "https://github.com/solana-foundation/pay-kit", branch = "main", default-features = false, features = [ | ||
| # builders). Tracks the confidential-transfer + solana-4.0 branch (pay-kit | ||
| # PR #181) until it merges to main. | ||
| solana-mpp = { git = "https://github.com/solana-foundation/pay-kit", branch = "feat/confidential-transfers", default-features = false, features = [ | ||
| "client", | ||
| "server", | ||
| "confidential", | ||
| ] } | ||
|
|
||
| # Solana x402 (lives in the pay-kit workspace alongside solana-mpp) | ||
| solana-x402 = { git = "https://github.com/solana-foundation/pay-kit", branch = "main", package = "solana-x402", default-features = false, features = [ | ||
| solana-x402 = { git = "https://github.com/solana-foundation/pay-kit", branch = "feat/confidential-transfers", package = "solana-x402", default-features = false, features = [ | ||
| "client", | ||
| ] } | ||
|
|
||
|
|
@@ -126,8 +127,9 @@ solana-signature = "3.2" | |
| solana-system-interface = "2.0" | ||
| solana-transaction = "3.1" | ||
|
|
||
| # Local override for end-to-end testing of unreleased pay-kit changes. | ||
| # Uncomment to redirect both crates to a local pay-kit checkout. | ||
| # [patch."https://github.com/solana-foundation/pay-kit"] | ||
| # solana-x402 = { path = "/Users/ludo/Coding/pay-kit/rust/crates/x402" } | ||
| # solana-mpp = { path = "/Users/ludo/Coding/pay-kit/rust/crates/mpp" } | ||
| # litesvm fork (solana-address pin loosened) so litesvm 0.13 — pulled by | ||
| # surfpool-sdk 1.4 — can coexist with the confidential proof crates (zk-sdk 7). | ||
| # Pending upstream PR; tracks the fork branch for now. | ||
| [patch.crates-io] | ||
| litesvm = { git = "https://github.com/lgalabru/litesvm.git", branch = "loosen-solana-address-constraint" } | ||
| litesvm-token = { git = "https://github.com/lgalabru/litesvm.git", branch = "loosen-solana-address-constraint" } | ||
|
Comment on lines
+133
to
+135
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
The
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Temporary dev shim: the fork only loosens litesvm's |
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both
solana-mppandsolana-x402are pinned tobranch = "feat/confidential-transfers"rather than a fixedrev =. The Cargo.lock does capture the commit hash at lock time, but anycargo updatewill silently pull whatever is at the HEAD of that branch at that moment — including force-pushed rewrites or last-minute breaking changes. Given the feature branch is still in review (PR #181), this is a concrete risk window. Consider locking to the current HEAD rev instead, e.g.,rev = "abc1234", until the upstream branch stabilises or merges.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intentional and temporary:
solana-mpp/solana-x402track pay-kit PR #181's feature branch until it merges, then move to a pinned release/rev. Noted in the Cargo.toml comment and the dev-shims section of confidential-transfers.md.