freighter
freighter copied to clipboard
getIconUrlFromIssuer should also return homeDomain
In the new design, we often need to show the homeDomain
alongside the asset icon. Getting the homeDomain
is part of the getIconUrlFromIssuer flow, but it is not cached anywhere because it was never needed before. Make getIconUrlFromIssuer both cache and return homeDomain
This may not be needed when we switch to using the new wallet-sdk
From @leighmcculloch:
We should be cautious displaying the domain from the asset issuers Stellar account home domain field, because anyone can put any domain into their accounts home domain. For example, I could put stellar.org in my personal domain. Or a wallet may decide to put the services domain into the home domain field of all their users.
Do we verify that the Stellar account that has the domain in the home domain field is mentioned as a CURRENCY in the stellar.toml of the domain to confirm there's a two-way relationship?
We started defining a protocol for asset issuer home domain verification in SEP-32. SEP-32 defines a process to verify the two-way relationship and may be useful for Freighter to implement.