Skip to content

Fix: sourcify verification for Ronin networks#176

Merged
huyhuynh3103 merged 4 commits into
mainnetfrom
fix/sourcify-v1-v2-verification
Mar 23, 2026
Merged

Fix: sourcify verification for Ronin networks#176
huyhuynh3103 merged 4 commits into
mainnetfrom
fix/sourcify-v1-v2-verification

Conversation

@huyhuynh3103
Copy link
Copy Markdown
Contributor

@huyhuynh3103 huyhuynh3103 commented Jan 12, 2026

  • Ronin's Sourcify uses V1 API (deprecated) which is incompatible with Foundry 1.5.x+ that uses V2 API by default
  • For ronin-mainnet/ronin-testnet: use custom post-deployment verification via Ronin's Sourcify V1 API (/verify endpoint with metadata.json)
  • For other networks: use Foundry's built-in --verifier sourcify (V2 API)
  • Add fallback check via /check-by-addresses endpoint to confirm verification

Description

Checklist

  • I have clearly commented on all the main functions following the NatSpec Format
  • The box that allows repo maintainers to update this PR is checked
  • I tested locally to make sure this feature/fix works

- Ronin's Sourcify uses V1 API (deprecated) which is incompatible with
  Foundry 1.5.x+ that uses V2 API by default
- For ronin-mainnet/ronin-testnet: use custom post-deployment verification
  via Ronin's Sourcify V1 API (/verify endpoint with metadata.json)
- For other networks: use Foundry's built-in --verifier sourcify (V2 API)
- Add fallback check via /check-by-addresses endpoint to confirm verification

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@huyhuynh3103 huyhuynh3103 force-pushed the fix/sourcify-v1-v2-verification branch from ee3997c to 4276306 Compare March 23, 2026 06:36
@huyhuynh3103 huyhuynh3103 changed the title Fix Sourcify verification for Ronin networks Fix: sourcify verification for Ronin networks Mar 23, 2026
@huyhuynh3103 huyhuynh3103 merged commit cf307eb into mainnet Mar 23, 2026
1 check passed
@huyhuynh3103 huyhuynh3103 deleted the fix/sourcify-v1-v2-verification branch March 23, 2026 08:57
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