Project

General

Profile

CdmAuthorisationAndAccessControl » History » Version 62

Andreas Kohlbecker, 10/11/2019 09:08 AM

1 56 Andreas Kohlbecker
# CDM authorisation and access control
2 2 Andreas Kohlbecker
3
It is evident that the cdm library needs authorisation and access control. There are library methods which need to protected from unauthorised execution and there is also the data which is exposed by the library. Not all data should be visible to every user so a [[HibernateSpringAndRowlevelSecurity|row level access control]] is needed.
4
5 57 Andreas Kohlbecker
Further information can be found in:
6
7
* [[WorkshopRightsAndRoles2017-11| Workshop Rights & Roles 2017-11]]
8
9 2 Andreas Kohlbecker
10 4 Andreas Kohlbecker
_This page only is a brief note for now, compiled by a lot of copy paste from the java doc!_
11 52 Andreas Kohlbecker
12 56 Andreas Kohlbecker
{{>toc}}
13 18 Andreas Kohlbecker
14 59 Andreas Kohlbecker
## Roles and CdmAuthorities
15 19 Andreas Kohlbecker
16
#### Roles
17 20 Andreas Kohlbecker
18 52 Andreas Kohlbecker
Roles are defined in two places:
19 19 Andreas Kohlbecker
20 52 Andreas Kohlbecker
1. eu.etaxonomy.cdm.persistence.hibernate.permission.Role
21
  * ROLE_ADMIN
22
  * ROLE_PROJECT_MANAGER
23 60 Andreas Kohlbecker
  * ROLE_USER_MANAGER (permission to modify User, Group, GrantedAuthority)
24
  * ROLE_PUBLISH (permission to modify the publish flag of Taxon, ...)
25
  * ROLE_REMOTING (permission to use the /remoting/** webservice, that is to connect the Taxeditor to a cdm instance, see #7972)
26 1 Andreas Kohlbecker
1. eu.etaxonomy.cdm.remote.config.MultiWebSecurityConfiguration
27 54 Andreas Kohlbecker
 * ROLE_MANAGE_CLIENT: see [Web service access control](#Web-service-access-control)
28 20 Andreas Kohlbecker
29
#### CdmAuthority
30
31 21 Andreas Kohlbecker
A `CdmAuthority` consists basically of two parts which are separated by a dot character '.'.
32 20 Andreas Kohlbecker
33 21 Andreas Kohlbecker
* permissionClass: an `CdmPermissionClass` instance with represents a cdm type or a part of the cdm type hierarchy. The className is always represented as an upper case string.
34
* property: The `CdmAuthority` only applies to instances which satisfy the specified property. Interpretation is up to type specific voters.
35
* operation: a string which specifies a `Operation` on that set of cdm types
36 1 Andreas Kohlbecker
* targetUuid: The operation may be restricted to a specific cdm entity by adding the entity uuid to the operation. The uuid string is enclosed in curly brackets '{' , '}' and appended to the end of the operation.
37 20 Andreas Kohlbecker
38
Examples for permissionStrings:
39
40
~~~
41 41 Andreas Kohlbecker
 TAXONBASE.[CREATE]
42
 TAXONBASE.[READ]
43
 TAXONBASE.[UPDATE]
44
 TAXONBASE.[DELETE]
45
 TAXONBASE.[DELETE,CREATE,UPDATE,READ]
46
 DESCRIPTIONBASE.UPDATE]
47
 DESCRIPTIONELEMENTBASE(Ecology).UPDATE]
48 1 Andreas Kohlbecker
 TAXONNODE.UPDATE{20c8f083-5870-4cbd-bf56-c5b2b98ab6a7}
49
~~~
50 21 Andreas Kohlbecker
51 24 Andreas Kohlbecker
in contrast to CdmAuthorities there are more general role like the following listed below, all these roles are having the  role prefix `'ROLE_'` which is defined in the String Security RoleVoter class:
52
53
~~~
54
 ROLE_ADMIN
55
 ROLE_USER_MANAGER
56
~~~
57
58 43 Andreas Kohlbecker
### Permission Groups
59
60
Permission groups can usually created as needed. The group name should reflect semantics the nature of the permission group.
61
62
The cdm provides default permission groups which are created by the `FirstDataInserter` in case they do not yet exist. See #4082 (implement default permission groups) for implementation details. Currently the following default permission groups exist: 
63
64 45 Andreas Kohlbecker
1. *Editor*: Contains all Authorities which are required to be granted for general editing, except the Authority for a specific taxonomical group. In order to actually enable an Editor to edit something it must be associated with a TAXONNODE Authority, e.g: @TAXONNODE.[CREATE,UPDATE,DELETE,READ]{...@}. Users which belong to this permission group are prohibited from toggling the publish-flag and from changing or deleting references and authors, but are able to create new ones. Once a 'Editor' has created a new Reference or Author the new entity is considered being still in the process of being created for some time even if the entity has already been saved to the database. See #4305 (newly created entities must stay editable even if a user only has the permission to create them) for implementation details on this feature.
65 43 Andreas Kohlbecker
66
~~~
67
REFERENCE.[CREATE,READ]
68
TAXONNAMEBASE.[CREATE,READ,UPDATE]
69
TEAMORPERSONBASE.[CREATE,READ]
70
TAXONBASE.[CREATE,UPDATE,DELETE,READ]
71 1 Andreas Kohlbecker
DESCRIPTIONBASE.[CREATE,UPDATE,DELETE,READ]
72 43 Andreas Kohlbecker
DESCRIPTIONELEMENTBASE.[CREATE,UPDATE,DELETE,READ]
73 60 Andreas Kohlbecker
ROLE_REMOTING
74 43 Andreas Kohlbecker
~~~
75 44 Andreas Kohlbecker
1. *ProjectManager*: A permission group which enables a User to edit and delete entities which are of central importance for a whole project. Users which belong to this permission group are granted to edit and delete taxa, names, references, authors, terms and can toggle the publish-flag on taxa, speciemens and occurrences.
76 43 Andreas Kohlbecker
77
~~~
78
REFERENCE.[UPDATE,DELETE]
79
TAXONNAMEBASE.[DELETE]
80
TEAMORPERSONBASE.[UPDATE,DELETE]
81
ROLE_PROJECT_MANAGER
82
~~~
83
84 21 Andreas Kohlbecker
### Authorisation control
85
86
Different approaches of authorisation control need to be used in order to implement all kinds of CRUD permissions:
87
88
* service layer: method `@PreAuthorize` annotations with Spring EL, e.g. UserService:
89
90
~~~
91 1 Andreas Kohlbecker
@PreAuthorize("#username == authentication.name or hasRole('ROLE_ADMIN')")
92 21 Andreas Kohlbecker
public void changePasswordForUser(String username, String newPassword) {
93 1 Andreas Kohlbecker
...
94 21 Andreas Kohlbecker
~~~
95
* UPDATE and CREATE of cdm instances is protected by a Hibernate interceptor, the `CdmSecurityHibernateInterceptor@. This interceptor implements the  @onSave()` and `onFlushDirty` methods and thus can control creation and updating of cdm objects. Hibernate calls these methods for any entity which needs to be persisted even if a save or update operations causes a cascading save of objects connected to the object graph which is being saved. **TODO** implement DELETE (#3079)
96
* READ permissions can not be implemented using the `CdmSecurityHibernateInterceptor@. Reasons: @onLoad()` is called just before an object is initialized, so the object has no uuid or anything else with would be needed to decide on principals authorisation. A row based access control mechanism is required, see [[HibernateSpringAndRowlevelSecurity]]
97 4 Andreas Kohlbecker
98
99 22 Andreas Kohlbecker
### Authorisation evaluation
100
101
102 41 Andreas Kohlbecker
One of the central classes for the evaluation of Roles and CdmAuthorities is the `CdmPermissionEvluator` with implements the `org.springframework.security.access.PermissionEvaluator@. The method @public boolean hasPermission(Authentication authentication, Object targetDomainObject, Object permission)` will always return true if the authentication has the role `ROLE_ADMIN` otherwise decission making is delegated to a `AccessDecisionManager` which should be `UnanimousBasedUnrevocable@. This @AccessDecisionManager` polls all configured AccessDecisionVoters grants access if only grant (or abstain) votes were received. The `AccessDecisionManager` then asks the set of plugged in **@AccessDecisionVoter@s*. The cdm specific voters are found in the package eu.etaxonomy.cdm.persistence.hibernate.permission.voter. For actual configuration details please see the *persistence_security.xml** , here is an example: 
103 22 Andreas Kohlbecker
104 1 Andreas Kohlbecker
~~~
105 41 Andreas Kohlbecker
<bean id="accessDecisionManager" class="org.springframework.security.access.vote.UnanimousBasedUnrevocable">
106 22 Andreas Kohlbecker
        <property name="decisionVoters">
107
            <list>
108
                <bean class="eu.etaxonomy.cdm.persistence.hibernate.permission.voter.GrantAlwaysVoter" />
109
                <bean class="eu.etaxonomy.cdm.persistence.hibernate.permission.voter.TaxonNodeVoter" />
110
                <bean class="eu.etaxonomy.cdm.persistence.hibernate.permission.voter.TaxonBaseVoter" />
111
                <bean class="eu.etaxonomy.cdm.persistence.hibernate.permission.voter.DescriptionBaseVoter" />
112
                <bean class="eu.etaxonomy.cdm.persistence.hibernate.permission.voter.DescriptionElementVoter" />
113
            </list>
114
        </property>
115
</bean>
116
117
<!--
118 41 Andreas Kohlbecker
    CdmPermissionEvaluator.hasPermissions() evaluates the CdmPermissions like TAXONNODE.[UPDATE]{20c8f083-5870-4cbd-bf56-c5b2b98ab6a7}
119 22 Andreas Kohlbecker
-->
120
<bean id="cdmPermissionEvaluator" class="eu.etaxonomy.cdm.persistence.hibernate.permission.CdmPermissionEvaluator">
121
    <property name="accessDecisionManager" ref="accessDecisionManager" />
122
</bean>
123
~~~
124 26 Andreas Kohlbecker
125
#### AccessDecisionVoters
126
127
As briefly described above, the AccessDecisionVoters are crucial for finding a decision on whether a user is allowed to perform an operation on a specific cdm object or class of cdm objects. The operations are the so called CRUD operations. This acronym refers to Create, Read, Update, Delete. 
128 1 Andreas Kohlbecker
129
A user can have one or multiple granted authorities, for more information on granted authorities please refer to the paragraph [[CdmAuthorisationAndAccessControl#CdmAuthority|CdmAuthority]] above. All voters will cast votes on all granted authorities a user has.
130 35 Andreas Kohlbecker
131 1 Andreas Kohlbecker
In the following we will just use the term voter as a short form for AccessDecisionVoter. 
132
133
Voters can cast three different vote:
134
135
* Allow
136
* Deny 
137
* Abstain
138
139
In order to make a access decision the AccessDecisionVoters are asked sequentially to cast their vote. If at least one voter casts _Deny_ the the whole operation will be denied. This behaviour is guaranteed by the `GrantAlwaysVoter` which will always cast an _Allow_.
140
141
All AccessDecisionVoters are responsible for a specific class of data entities and thus it will only cast an _Allow_ or _Deny_ vote when an operation on an object of this class is to be performed. If a voter is not responsible it will _Abstain_. If a voter "feels" not responsible it casts an _Abstain_. As a consequence of this concept everything is allowed unless a voter is responsible for a specific object class. Only in this case there is the potential that an operation will be denied.
142
143
Generally each of the voters will check if the user has a granted authority which matches the the operation and object. Voters can also perform additional checks (see TaxonNodeVoter, DescriptionElementVoter) which can take related objects into account (parent taxon nodes) or specific properties of of Objects (feature of description element).
144
145
List of existing AccessDecisionVoters in the order of their execution:
146
  
147
1. TaxonNodeVoter: 
148
  * responsible for TaxonNode
149
  * allows if voter is responsible for the object class in question and
150
    1. if the user has a granted authority for that specific object or for all objects of this class which matches the operation to be performed
151 61 Andreas Kohlbecker
    1. or if the user has a matching granted authority for any of the parent taxon nodes in the classification (e.g.: `TAXONNODE.UPDATE{20c8f083-5870-4cbd-bf56-c5b2b98ab6a7}`)
152 1 Andreas Kohlbecker
  * denies otherwise
153
1. TaxonBaseVoter:  
154
  * responsible for Taxon, Synonym
155
  * allows if voter is responsible for the object class in question and
156 61 Andreas Kohlbecker
    1. if the user has a granted authority for that specific object or for all objects of this class which matches the operation to be performed (e.g.: `TAXONBASE.UPDATE`)
157 1 Andreas Kohlbecker
  * denies otherwise
158
1. DescriptionBaseVoter:  
159
  * responsible for TaxonDescription, SpecimenDescription
160
  * allows if voter is responsible for the object class in question and
161 61 Andreas Kohlbecker
    1. if the user has a granted authority for that specific object or for all objects of this class which matches the operation to be performed (e.g.: `DESCRIPTIONBASE.UPDATE`)
162 1 Andreas Kohlbecker
  * denies otherwise
163
1. DescriptionElementVoter:  
164
  * responsible for all types of DescriptionElements
165
  * allows if voter is responsible for the object class in question and
166
    1. if the user has a granted authority for that specific object or for all objects of this class which matches the operation to be performed
167 61 Andreas Kohlbecker
    1. if the the operation to be preformed is specific to the Feature of the Description Element and if the _Property_ of one of the users granted authorities matches this Feature. (e.g.: `DESCRIPTIONELEMENTBASE(Ecology).[UPDATE]`)
168 1 Andreas Kohlbecker
  * denies otherwise
169
170 58 Andreas Kohlbecker
## Cdm Entity access control
171
172
### Type bases access control
173
174
This is defined through the general *CdmAuthotities* which are not referring to a specific entity uuid.
175
176
### Per entity access control
177
178
This is defined through the entity specific *CdmAuthotities* which referring to a specific entity by the uuid.
179
180
### Extended create permission problem
181
182
for e general discussion see #4305 (newly created entities must stay editable even if a user only has the permission to create them)  for details regarding the current implementation see #6867 (explicitely assign and revoke UPDATE & DELETE permission per enitity in the registration workflow). Problems with the current implementation: #7147, ...
183
184
185 1 Andreas Kohlbecker
## Web service access control
186
187
### Authentication
188
189
All web service end points which require authentication support http basic.
190
191
There are two sources of users:
192
193 56 Andreas Kohlbecker
**CDM users**: 
194
195
The users stored in the cdm database
196
197
**Global managing users**: 
198
 
199
Users stored in the Managing users properties file (`$HOME/.cdmLibrary/managing-users.properties`). This is a java properties file to populate the InMemoryUserDetailsManager in any of the cdm-remote instances with special global management users which are granted to access special web services. Among these are the /manage/ web services and those triggering long running tasks. Global management users have the role **ROLE_MANAGE_CLIENT** and will be available in each of the cdm-remote instances. Changes made to this file are applied after restarting a cdm instance. For implementation details please refer to #6248.
200
201
Web service endpoints accessible to global managing users:
202
203
* `/manage/**`
204
* `/**description/accumulateDistributions`
205 52 Andreas Kohlbecker
206
### Authorization
207
208
1. Classical spring security authorization as described above
209 53 Andreas Kohlbecker
1. OAuth2: see #6118 *evaluate spring-security-auth2 and spring-cloud-security as a framework for OAuth2*
210 52 Andreas Kohlbecker
211
## Use cases
212
213
 A. a specific **classification sub tree** is **publicly visible** that is it is _published_
214
 B. a specific **classification sub tree** must not be **publicly visible** in the data portal and thus must be also hidden in the web service responses
215
 C. a specific **classification sub tree** is only visible for users which have a specific **role** but the user is not granted to **edit** anything in/below it
216
 D. A user is only granted to **edit Descriptions** 
217
 E. A user is only granted to **edit DescriptionElements** of a specific _Feature_ 
218
 F. A user is only granted to **edit structured Descriptions** 
219
 G. Combinations of B, C and D, E must be possible
220
 H. Only users with the **roles** *Admin* or **Usermanager** or the user in question it self (if currently authenticated) are allowed to execute **change password** 
221
 I. Only users with the **roles** *Admin* or **Usermanager** are allowed to create or **edit** new users
222
223
~~~
224
<code class="rst">
225
Tabular summary of above use cases translated into roles:
226
227
===========  =========================  ======================================  ===============  =============================
228
\                                                                               what to protect
229
-----------  -------------------------  --------------------------------------  ----------------------------------------------
230
Usecase      Role                       authority string                        Entity            ServiceMethod
231
===========  =========================  ======================================  ===============  =============================
232
A.           Anonymous                  TaxonNode.[READ]{uuid}                     ...
233
B.           TaxGroupX_User             TaxonNode.[READ]{uuid}                     ...
234
C.           TaxGroupX_Editor           TaxonNode.[UPDATE]{uuid}                   ...
235
D.           DescriptionEditor		Description.[UPDATE]			 ...					
236
E.           DescriptionEditor		DescriptioElement(Ecology).[UPDATE]        ...					
237
H.           Admin, Usermanager                                                                  UserService.changePassword()
238
===========  =========================  ======================================  ===============  =============================
239
240
*Anonymous* means not authenticated  
241
242
~~~
243
244
 **Special cases:** 
245 1 Andreas Kohlbecker
246 62 Andreas Kohlbecker
* **`TaxonNames`** can potentially be shared between different taxa, thus a situation may occur where a user has grants to edit taxon A but not for taxon B, but both taxa are sharing the same name. How will we handle this situation, should the name be cloned when the user starts editing taxon A, so that taxon A has another name entity than taxon B after the user saved the latest changes?
247
* The same problem as described above for TaxonNames also accounts for *`References`*, but in this case the problem is more severe since references are very often part of multiple taxon names.