0xjac / ERC777

New Standard for Ethereum Token
Mozilla Public License 2.0
128 stars 70 forks source link

Implement `revokeAllOperators` #35

Closed 0xjac closed 6 years ago

0xjac commented 6 years ago

For convenience and security reasons, a new function should be added:

function revokeAllOperators() public;

This method revokes all the operators for msg.sender.

Quazia commented 6 years ago

The only way I could see for revoking all authorized operators would be to track a list of authorized operators. Even if a list of authorized operators was kept there may be situations where iterating through the full list of authorized operators would cause an out of gas exception. Any ideas on how to get around this?

0xjac commented 6 years ago

Yes we do, it's a bit lengthy to explain here but I will push an implementation for it in the next few days. I'll ping you so you can take a look. Any feedback is welcome!

0xjac commented 6 years ago

After some thinking and discussing with a few people this will not be implemented.