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

Distinguish Banker From Client for Access Control #1

Open
VRamakrishnaSG opened this issue May 21, 2018 · 1 comment
Open

Distinguish Banker From Client for Access Control #1

VRamakrishnaSG opened this issue May 21, 2018 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@VRamakrishnaSG
Copy link
Contributor

In the organization structure of the initial version of the network, a single organization encompasses banker and client for the importing as well as the exporting side (thereby conflating the role of Importer and ImporterBank, and Exporter and ExporterBank, respectively.)

Currently, the access control mechanisms that have been built cannot distinguish these two roles, neither in chaincode nor in the application.

There are two ways to fix this. We probably should implement both:

  1. Have the organization MSP issue certificates with different attributes to banker and client. Then change chaincode logic to check for the right attributes before executing the transaction.
  2. Have the web service (in application) associate different user IDs with different roles, and control access to chaincode functions at that level. We could also consider adding a role parameter to the chaincode arguments list (we would have to augment the chaincode too.)

Anyone want to take a shot at this? I don't think this is very urgent, as we can discuss access control theoretically in the book and even leave the implementation as a reader exercise. In an case, the code can have a (parallel) life of its own, independent of the book. We'll just need to make clear what version of the code the book is referring to.

@VRamakrishnaSG VRamakrishnaSG added the enhancement New feature or request label May 21, 2018
@VRamakrishnaSG VRamakrishnaSG self-assigned this Jun 5, 2018
@solithejunior
Copy link

respectly i wanna from you all that participating in logistic and solve many issues to deploy it with your experience,
so if you honour me my email is [email protected]

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
None yet
Development

No branches or pull requests

4 participants