Skip to content

fix(wnft): reject invalid signer addresses#1045

Open
modelsbridgeaicom-ship-it wants to merge 1 commit into
openmetaearth:mainfrom
modelsbridgeaicom-ship-it:codex/wnft-getsigners-panic-invalid-282
Open

fix(wnft): reject invalid signer addresses#1045
modelsbridgeaicom-ship-it wants to merge 1 commit into
openmetaearth:mainfrom
modelsbridgeaicom-ship-it:codex/wnft-getsigners-panic-invalid-282

Conversation

@modelsbridgeaicom-ship-it

Copy link
Copy Markdown
Contributor

Addresses WNFT-NEW-002 in #282.

Summary

  • updates WNFT MsgNewClass, MsgMintNFT, and MsgSend GetSigners to use MustAccAddressFromBech32
  • prevents invalid signer addresses from silently returning an empty/zero signer
  • adds focused unit coverage for invalid signer addresses on all three WNFT messages

Tests

  • ....\tools\go\bin\go.exe test .\x\wnft\types -count=1 -v
  • git diff --check
  • git diff --cached --check

@koelzen koelzen added run-ci Trigger CI run and removed run-ci Trigger CI run labels Jun 10, 2026
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