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

Reinstate delegate calls (or something) #140

Open
nhynes opened this issue Jul 14, 2019 · 0 comments
Open

Reinstate delegate calls (or something) #140

nhynes opened this issue Jul 14, 2019 · 0 comments
Assignees
Labels
p:2 Priority: desired feature

Comments

@nhynes
Copy link
Contributor

nhynes commented Jul 14, 2019

One will notice that all calls are of the regular variety. Delegate calls are philosophically impure since every callee can inherit capabilities to the top-level caller. They're kind of useful, though, for keeping downstream code clean (e.g., for ERC20, the sender is always the account from which to transfer).

Ideally we'd just send caps individually, but this requires more thought. In the interim, we should just enable delegatecall (perhaps with a deprecation warning). Thank Ekiden for contract expiry 🙏

@nhynes nhynes self-assigned this Jul 14, 2019
@willscott willscott added the p:2 Priority: desired feature label Jul 15, 2019
dm4 pushed a commit to second-state/oasis-parity that referenced this issue Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p:2 Priority: desired feature
Projects
None yet
Development

No branches or pull requests

2 participants