Skip to content

feat: introduce sandbox environment support#385

Open
paolodamico wants to merge 2 commits into
mainfrom
sandbox
Open

feat: introduce sandbox environment support#385
paolodamico wants to merge 2 commits into
mainfrom
sandbox

Conversation

@paolodamico

@paolodamico paolodamico commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Fairly explanatory. Reference project linked in the code.


Note

Medium Risk
New public enum variant affects all clients via UniFFI; misconfigured sandbox could point at wrong contracts or attestation trust if other env matches were missed (this diff only updates the sites shown).

Overview
Adds a third BedrockEnvironment::Sandbox value (UniFFI-exposed) with wire string "sandbox" and docs describing it as an external TFH testing environment that still uses World ID staging protocol settings.

Sandbox is treated like staging everywhere this PR touches environment branching: Nitro enclave attestation uses staging PCR configs and the staging AWS Nitro root cert; world_gift_manager_address and test helper address_book_address resolve to the same contract addresses as staging.

Minor comment whitespace fixes in nitro_enclave/mod.rs only.

Reviewed by Cursor Bugbot for commit aed7f07. Bugbot is set up for automated code reviews on this repo. Configure here.

@brian-terczynski brian-terczynski left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but will let someone with more familiarity provide the approval. Thanks for the changes!

@brian-terczynski brian-terczynski left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bumping this to approval so we can move forward.

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.

2 participants