Skip to content

Validate send message amount#12

Open
giwaov wants to merge 1 commit into
canopy-network:mainfrom
giwaov:validate-send-message-amount
Open

Validate send message amount#12
giwaov wants to merge 1 commit into
canopy-network:mainfrom
giwaov:validate-send-message-amount

Conversation

@giwaov

@giwaov giwaov commented Jul 6, 2026

Copy link
Copy Markdown

Summary

  • reject built-in send message amounts that cannot be safely encoded as uint64 values
  • prevent protobufjs from coercing negative, fractional, or NaN amounts into different wire values
  • add protobuf coverage for valid and invalid send amounts

Tests

  • npx vitest run src/tests/protobuf.test.ts
  • npm run build

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