-
Notifications
You must be signed in to change notification settings - Fork 303
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
Add support for roles and permissions #344
Conversation
@pushpabrol can you check the missing test coverage? thanks for the pr 🎉 |
That's also possible to add permissions on Auth0 APIs? |
@luisrudge not in the role. I need to add the permission in a custom API. Add the permission to the role was enough to use the application and give the right permissions to the users, but on Postman tests, when I get the token, I can only get permissions defined on API, since is a machine-to-machine flow. Today I can do using this way, but it's very time consuming and hard to maintain between the different environments. |
@luisrudge, I've founded how to do it: update the http://auth0.github.io/node-auth0/module-management.ManagementClient.html#updateResourceServer |
I agree with you that it's confusing that the frontend calls resource servers as APIs :( but I'm glad you found it! |
Changes
-Changed management/index.js
References
Testing
Add
test/management/roles.tests.js
and modifiedtest/management/users.tests.js
This change adds unit test coverage
This change adds integration test coverage
Checklist
I have read the Auth0 general contribution guidelines
I have read the Auth0 Code of Conduct
All existing and new tests complete without errors
All new tests complete without errors but some old tests seem to be erroring out