Skip to content

fix: validate tao ss58 checksums#312

Open
volcano303 wants to merge 1 commit into
entrius:testfrom
volcano303:fix/tao-ss58-validation
Open

fix: validate tao ss58 checksums#312
volcano303 wants to merge 1 commit into
entrius:testfrom
volcano303:fix/tao-ss58-validation

Conversation

@volcano303
Copy link
Copy Markdown

Summary

  • validate TAO addresses with Bittensor's SS58 checksum validator instead of regex-only shape checks
  • add regression coverage for checksum-mutated and base58-shaped invalid addresses

Issue

Closes #200

Validation

  • uv run ruff check allways/ neurons/ tests/test_scale.py
  • uv run pytest tests/test_scale.py::TestIsValidAddress -q
  • uv run pre-commit run --all-files
  • uv run pytest tests/

@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TAO receive address not SS58-checksum-validated — user typo sends funds irrecoverably

1 participant