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

Why is there a role field in DDB tenant table? #61

Open
ilove2git opened this issue Jun 17, 2021 · 1 comment
Open

Why is there a role field in DDB tenant table? #61

ilove2git opened this issue Jun 17, 2021 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@ilove2git
Copy link

when going through lab1 module, I see there's a field name 'role' whose sample value is tenantAdmin in tenant table. I'm confused about this field. From my point of view, I suppose it should only exist in Cognito User Pool as custom attribute such as tenantAdmin or tenantUser .etc. Please help correct me if wrong. thanks

@brtrvn
Copy link
Contributor

brtrvn commented Sep 22, 2021

You are correct. That is a bug. We do not need to store the authorization role in the tenant manager micro service's data (and the actual tenant registration micro service does not set a "role" attribute when you register using the sample app vs manually invoking the REST endpoint as we do in Lab 1).

@brtrvn brtrvn added bug Something isn't working good first issue Good for newcomers labels Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants