Electric-Coin-Company / zcash-android-wallet-sdk

Native Android SDK for Zcash
MIT License
5 stars 9 forks source link

Add `RustLayerException.ValidateAddressException` #1534

Open HonzaR opened 3 months ago

HonzaR commented 3 months ago

Is your feature request related to a problem? Please describe.

A few Synchronizer APIs throw RuntimeException, which is too generic and should be replaced with a more specific one.

Describe the solution you'd like

Replace RuntimeException exception with e.g. RustLayerException.ValidateAddressException for these Synchronizer APIs:

Alternatives you've considered

Also, ensure that all the underlying functions properly state the RuntimeException exception in their contract, i.e., in Backend and TypesafeBackend.

Additional context