zcash-android-wallet-sdk icon indicating copy to clipboard operation
zcash-android-wallet-sdk copied to clipboard

Add `RustLayerException.ValidateAddressException`

Open HonzaR opened this issue 7 months ago • 0 comments

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:

  • Synchronizer.isValidShieldedAddr
  • Synchronizer.isValidTransparentAddr
  • Synchronizer.isValidUnifiedAddr
  • Synchronizer.isValidTexAddr

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

HonzaR avatar Jul 23 '24 12:07 HonzaR