Skip to content

research: test nested approve authorization for upto - #240

Open
dedukpe wants to merge 5 commits into
accensa:mainfrom
dedukpe:research/upto-nested-approve
Open

research: test nested approve authorization for upto#240
dedukpe wants to merge 5 commits into
accensa:mainfrom
dedukpe:research/upto-nested-approve

Conversation

@dedukpe

@dedukpe dedukpe commented Aug 26, 2026

Copy link
Copy Markdown

Closes #62

Summary

Resolves #62 by experimentally testing whether a single Soroban authorization entry can cover an authorize call containing a nested SEP-41 token.approve sub-invocation.

What Was Tested

  • Construction of a nested Soroban authorization tree.
  • Signing and execution using a single payer authorization entry.
  • Resulting authorization-tree structure.
  • What the payer's signature commits to.
  • Token approval state after execution.
  • Nested versus separate invocation budget costs.
  • @x402/stellar validateAuthEntries behavior.
  • Wallet presentation where the environment allowed it.

Findings

The spike documents whether Soroban technically permits the construction separately from whether the current x402 Stellar facilitator/ecosystem considers the authorization acceptable.

ADR-002 §6.2 has been updated with the resulting evidence and recommendation.

Scope

This is an isolated research spike.

No production upto implementation or production contract authorization behavior was changed.

Validation

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • Dedicated nested-authorization spike tests

Evidence

The PR includes the reproducible experiment, authorization-tree analysis, budget comparison, and validateAuthEntries findings.

@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

@dedukpe Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

research(upto): empirically test whether one auth entry can cover a nested approve (ADR-002 §6.2)

1 participant