Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement revokeAllOperators #35

Closed
0xjac opened this issue Feb 23, 2018 · 3 comments
Closed

Implement revokeAllOperators #35

0xjac opened this issue Feb 23, 2018 · 3 comments
Assignees

Comments

@0xjac
Copy link
Owner

0xjac commented Feb 23, 2018

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

function revokeAllOperators() public;

This method revokes all the operators for msg.sender.

@0xjac 0xjac self-assigned this Feb 23, 2018
@Quazia
Copy link

Quazia commented Feb 23, 2018

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
Copy link
Owner Author

0xjac commented Feb 23, 2018

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
Copy link
Owner Author

0xjac commented Apr 30, 2018

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

@0xjac 0xjac closed this as completed Apr 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants