Skip to content

feat(wallet): let Safe users open the explorer as a Safe App - #816

Open
rickstaa wants to merge 1 commit into
fix/safe-transactionsfrom
feat/safe-app
Open

rickstaa wants to merge 1 commit into
fix/safe-transactionsfrom
feat/safe-app

Conversation

@rickstaa

@rickstaa rickstaa commented Sep 22, 2026

Copy link
Copy Markdown
Member

Adds the manifest and CORS header Safe{Wallet} needs to accept the explorer as a custom Safe App, and a Safe connector (listed only inside an iframe) so the explorer opens already connected as the Safe. The connector only trusts app.safe.global as the parent page, so another site embedding the explorer cannot fake the Safe connection. Safes connected from outside Safe{Wallet} get a short hint under the transaction dialog to open the explorer as a Safe App instead.

Stacked on #815; retarget to main once that merges.

Closes #814.

@vercel

vercel Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
explorer-arbitrum-one Ready Ready Preview Sep 22, 2026 1:44pm UTC

Request Review

Only app.safe.global is trusted as the parent page, so a site embedding
the explorer cannot fake the Safe connection.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

This branch was successfully deployed

1 active deployment
Preview 2923e6e5 Deployed Sep 22, 2026 by vercel[bot]
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