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

Config env ALLOWED_ADMINS_LIST #812

Open
alexcos20 opened this issue Jan 22, 2025 · 0 comments
Open

Config env ALLOWED_ADMINS_LIST #812

alexcos20 opened this issue Jan 22, 2025 · 0 comments
Assignees
Labels
Type: Enhancement New feature or request

Comments

@alexcos20
Copy link
Member

We already have ALLOWED_ADMINS env, which decides which addresses are allowed admins.

Managing those address (add/remove) is cumbersome, cause it needs a restart of the node. But we have accessLists, so :

  • add ALLOWED_ADMINS env variable, which looks like
{
 "1": ["0x123","0x124"],
 "2": ["0x345"]
}

which means:

  • for chainId 1 , call "balanceOf(ADDRESS_TO_CHECK)" on contracts 0x123 and 0x124
  • for chainId 2 , call "balanceOf(ADDRESS_TO_CHECK)" on contract 0x345

This way, node owner can use accessList contracts to define list of addresses.

Ocean-node will check both ALLOWED_ADMINS and ALLOWED_ADMINS_LIST (if they are defined) and if an address is a match on any , then it's allowed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants