[MGNLREST-110] Implement improved REST roles Created: 04/Sep/17 Updated: 24/Oct/17 Resolved: 11/Sep/17 |
|
| Status: | Closed |
| Project: | Magnolia REST Framework |
| Component/s: | delivery, integration |
| Affects Version/s: | None |
| Fix Version/s: | 2.0 |
| Type: | Improvement | Priority: | Major |
| Reporter: | Mikaël Geljić | Assignee: | Dai Ha |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 0d | ||
| Time Spent: | 2d 7h | ||
| Original Estimate: | 2.5d | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||||||
| Epic Link: | REST Headless bundle | ||||||||||||||||||||||||||||||||
| Sprint: | Saigon 112, Saigon 113 | ||||||||||||||||||||||||||||||||
| Story Points: | 3 | ||||||||||||||||||||||||||||||||
| Description |
|
Implement new roles, as per concept - REST Permissions. The basic proposal is three roles covering different scenarios: rest-admin, rest-anonymous, and rest-editor (the latter renamed from current rest role). e.g. for rest-admin (Currently, most roles seem to use description as full name, but let's get this better for rest) |
| Comments |
| Comment by Dai Ha [ 06/Sep/17 ] |
|
As document mentions about renaming current 'rest' role to new 'rest-editor', this should be done in version handler but it causing other modules bootstrap errors. Affected modules so far: magnolia-rest-services Suggested solutions are either:
mgeljic could you please give your opinions with those above issues? |
| Comment by Mikaël Geljić [ 07/Sep/17 ] |
|
| Comment by Mikaël Geljić [ 07/Sep/17 ] |
|
I also realized a couple omissions in the concept (updated now):
As for the task updating security-base role, let's keep it untouched, should be fine. |
| Comment by Dai Ha [ 12/Sep/17 ] |
|
verified with magnolia-enterprise-pro-demo-bundle-5.6-20170911.202509-109-tomcat-bundle. |