privacy warning: t-addr

[TalDerei]

adds a privacy warning to the approval dialog if the Ics20Withdrawal message includes a transparent address (t-addr).

CI pipeline is currently failing because this depends on penumbra-zone/web#1950 and requires bumping package dependencies.

---

[TalDerei]

need to truncate IBC-address rendering which is currently getting cut-off

---

---

updated

[hdevalence]

Shouldn't the updated rendering show "use transparent address" rather than "use compat address"?

[TalDerei]

Shouldn't the updated rendering show "use transparent address" rather than "use compat address"?

yes, updating approval dialogue to reflect this.

a t-addr has an alternate encoding that is parsed into an Address. The current Ics20WithdrawalComponent in the approval dialogue displays that Address. Should it instead display the tpenumbra encoding format? cc @erwanor

[erwanor]

I am not going to block on it, but if we set the transparent flag on, I think it makes sense to render the withdrawal component with a return address encoded as a transparent address.

We do this by adding a PENUMBRA_BECH32_TRANSPARENT_ADDRESS_PREFIX with helpers to the web package @penumbra-zone/bech32. Then, we should be able to display the transparent encoding of an Address. This is how it's done in core: https://github.com/penumbra-zone/penumbra/blob/020986747245270a00d039306458be9e7f1f33f5/crates/core/keys/src/address.rs#L232

[erwanor]

LGTM, on green CI, with requested web changes (other pr) that should make it easy to do taddr rendering in the withdrawal component

[TalDerei]

We do this by adding a PENUMBRA_BECH32_TRANSPARENT_ADDRESS_PREFIX with helpers to the web package @penumbra-zone/bech32

renders the proper address format in the approval dialogue