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

base-transactions: add authorizedRegistrators property to RegisterCollection transaction #27

Closed
kristjank opened this issue Jun 4, 2020 · 0 comments · Fixed by #32
Closed
Assignees
Labels
enhancement New feature or request
Projects

Comments

@kristjank
Copy link
Contributor

kristjank commented Jun 4, 2020

Add allowed registrator property on the plugin and not on the blockchain level. This is for the intermediate term, as it helps us to centralize who is allowed to register new collections of assets on the blockchain.

{
    "authorizedRegistrators": ["PK1", "PK2",...."PKn"]
}

default value is [] - meaning anyone can send the transaction to the blockchain.

Conditions:

  • if value in [""] - only allow PK in list to sendTransactions
  • if value === [] - anyone can send transaction

Note

This is not an optimal solution, as it can affect the bootstrap process. We will address this in a separate issue.

@kristjank kristjank assigned kristjank and amacar and unassigned kristjank Jun 4, 2020
@kristjank kristjank added the enhancement New feature or request label Jun 4, 2020
@kristjank kristjank added this to In progress in NFT Jun 4, 2020
NFT automation moved this from In progress to Done Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
NFT
  
Done
2 participants