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

Registry upgrading #206

Closed
pabloruiz55 opened this issue Aug 22, 2018 · 0 comments
Closed

Registry upgrading #206

pabloruiz55 opened this issue Aug 22, 2018 · 0 comments

Comments

@pabloruiz55
Copy link
Contributor

pabloruiz55 commented Aug 22, 2018

  • All registry upgrades (STR, TR & MR) will be done in-place using a proxy / delegate call pattern.
  • This means that anyone using these contracts (e.g. SecurityToken or Modules) has no control over whether they get the upgrade or not.
  • As a result of this, any dependencies on these contracts are subject to breaking (i.e. if Polymath self-destructs the contract), which implies no existing functionality should have any dependencies on the registry contracts.
  • AFAICT this is true for all of our modules and security token. The security token has a dependency on the module registry for adding new modules, but this is fine as it doesn't contravene the aims above which relate to existing functionality, not new modules.
  • Registries do have interdependencies (e.g. module registry depends on STR, STR depends on TR), so when upgrading registries we'll need to make sure they are upgraded in a consistent manner.
@pabloruiz55 pabloruiz55 changed the title Registry upgrading: - All registry upgrades (STR, TR & MR) will be done in-place using a proxy / delegate call pattern. Registry upgrading: All registry upgrades (STR, TR & MR) will be done in-place using a proxy / delegate call pattern. Aug 22, 2018
@pabloruiz55 pabloruiz55 changed the title Registry upgrading: All registry upgrades (STR, TR & MR) will be done in-place using a proxy / delegate call pattern. Registry upgrading Aug 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Upgradeability
Awaiting triage
Development

No branches or pull requests

2 participants