Keeper-Wallet / Keeper-Wallet-Extension

Your entry point to the Waves blockchain and Waves-powered web services
https://keeper-wallet.app
Other
67 stars 46 forks source link

Revise Permissions Control #93

Closed bravetarget closed 2 years ago

bravetarget commented 4 years ago

Аbstract With the ability for a user to block out an entire domain, an accidental rejection on the initial auth leads to an undesirable UX. Especially for users unfamiliar with Waves Keeper.

Motivation and Purposes It'd be nice if we did not have to teach users how to re-enable permissions for a domain. This seems like an unnecessarily overreaching security feature and ultimately creates room for user error rather than protecting them. Consider these points:

Solution

Backwards Compatibility This would be cleanly backwards compatible, and improve the UX for dapps

Examples and Implementation Other browser wallets experience this same problem. However, I see this as a trait of nascent software rather than necessary security features.

msmolyakov commented 2 years ago

I apologize that there was no response to you for so long. And I am grateful to you for your ideas, we will take that into account.

Btw, we've already simplified the process a little at #115 - if initial auth is rejected, the domain is not added to the blacklist, so later it can resend the request. In the near future we will continue to make the process of using a Keeper more convenient for all the parties.