Page MenuHomePhabricator

[ecashaddrjs] Support decoding prefixless addresses if checksum is correct
ClosedPublic

Authored by bytesofman on Mar 27 2023, 12:56.

Details

Summary

T3039

Depends on D13474

Test Plan

Review change to function and new unit test. npm test and all pass.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien added a subscriber: Fabien.
Fabien added inline comments.
web/ecashaddrjs/src/cashaddr.js
88 ↗(On Diff #38918)

Nit: you could also print the list of supported prefixes and hint to not use a prefixless address if the prefix is not in this list

This revision is now accepted and ready to land.Mar 27 2023, 15:26

Print list of valid prefixes on prefixless error