Skip to content

docs(ethereum): clarify on-chain validation of price update payloads - #3852

Open
Ysh204 wants to merge 2 commits into
pyth-network:mainfrom
Ysh204:docs/clarify-payload-validation
Open

docs(ethereum): clarify on-chain validation of price update payloads#3852
Ysh204 wants to merge 2 commits into
pyth-network:mainfrom
Ysh204:docs/clarify-payload-validation

Conversation

@Ysh204

@Ysh204 Ysh204 commented Jun 29, 2026

Copy link
Copy Markdown

Summary

Fixes #3653 by clarifying how price update payloads are validated during Ethereum integrations.

Changes

  • Add warning notes to the Ethereum contracts, JavaScript SDK, and Solidity SDK READMEs.
  • Clarify that payload integrity and source validation occur on-chain during transaction execution.
  • Document that malformed payloads may revert with InvalidWormholeVaa() or InvalidUpdateDataSource().
  • Recommend obtaining payloads from trusted providers (such as Hermes) and simulating transactions before broadcasting.

Fixes #3653


Open in Devin Review

@vercel

vercel Bot commented Jun 29, 2026

Copy link
Copy Markdown

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

4 Skipped Deployments
Project Deployment Actions Updated (UTC)
entropy-explorer Skipped Skipped Jun 29, 2026 5:50am
insights Skipped Skipped Jun 29, 2026 5:50am
proposals Skipped Skipped Jun 29, 2026 5:50am
staking Skipped Skipped Jun 29, 2026 5:50am

Request Review

@vercel
vercel Bot temporarily deployed to Preview – staking June 29, 2026 05:50 Inactive
@vercel
vercel Bot temporarily deployed to Preview – insights June 29, 2026 05:50 Inactive
@vercel
vercel Bot temporarily deployed to Preview – proposals June 29, 2026 05:50 Inactive
@vercel
vercel Bot temporarily deployed to Preview – entropy-explorer June 29, 2026 05:50 Inactive
@vercel

vercel Bot commented Jun 29, 2026

Copy link
Copy Markdown

@Ysh204 is attempting to deploy a commit to the Pyth Network Team on Vercel.

A member of the Team first needs to authorize it.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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.

Docs/DX: Clarify on-chain validation and lack of pre-validation for price update payloads

1 participant