Skip to content
This repository has been archived by the owner on Dec 24, 2020. It is now read-only.

getVaultOwners() does not work #6

Open
aparnakr opened this issue Feb 22, 2020 · 0 comments
Open

getVaultOwners() does not work #6

aparnakr opened this issue Feb 22, 2020 · 0 comments

Comments

@aparnakr
Copy link
Contributor

getVaultOwners() may run out of gas if there are a large number of vault owners. This may make it difficult (or impossible if you don't have access to your own ethereum node with a custom configured high gas limit) to figure out who the vault owners are in order for someone trying to exercise to do the exercise.

It is recommended to query https://thegraph.com/explorer/subgraph/aparnakr/opyn to get all the vault owners instead of calling the function from the contract.

@aparnakr aparnakr changed the title getVaultOwners() may run out of gas getVaultOwners() does not work Apr 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant