-
Notifications
You must be signed in to change notification settings - Fork 872
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
Rights to read OUser for reader and writer #4906
Comments
Guys, Any thoughts on this? |
Contains hack for workaround issue orientechnologies/orientdb#4906
Guys, What do you think about idea to use JAAS approach for security in OrientDB? Layer1: If user is admin (has "bypass") action imidiatly granted Layers should be plugable not only on db side, but on app side. For example, application can insert additional layer for checking trial period and so on. Also it will help to solve this problem: if user wants access to his record - just grant it! Layers can have flags like in JAAS: required, requiresite, optional, sifficient and so on. And even more! Validation can be also treated as "security layer". Validation layer is interested in CREATE and UPDATE and grant operation only if document match validation criterias. Actually all of that can even be "packed" as RecordHooks. And I guess that redesign of this area has been already discussed several times. For example: #3429 |
👍 |
Hi guys, You have changed the permissions to access the OUser class but this change has not been reported in change log or into the migration page of the documentation (as far as I know) Here is how to reproduce the problem:
|
Here are the statements I used to fix the problem on my schema:
|
Guys,
I do know, that in 2.1.1 you disallow for reader's and writers to read OUser. Also I know that it can be reverted back by application. But please, could you make that in other way? User have to have access at least to his record. Otherwise it's the mess for web app which use OrientDB users for authentication. At least db.getUser() doesn't work properly and a lot of other not so good side-effects...
Unfortunately, we need to rollback to 2.1.0:( Hope that will be resolved in 2.1.2...
The text was updated successfully, but these errors were encountered: