[MGNLSSO-25] Write a UserManager to create users on the fly Created: 07/May/20 Updated: 07/Jul/20 Resolved: 15/May/20 |
|
| Status: | Closed |
| Project: | Single Sign On |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.0 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Maxime Michel | Assignee: | Maxime Michel |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoR: |
Empty
|
| Epic Link: | Magnolia SSO w/ pac4j |
| Description |
|
I'm working on a setup with external users. There are two scenarios there:
This is at least what Richard Unger mentions in the following page: http://magnolia.10149.n7.nabble.com/How-to-instantiate-a-User-to-call-MgnlContext-login-mgnlUser-td38527.html
I have gone with option 1. This setup is working well except in info.magnolia.task.persistence.TasksStoreImpl#prepareQueryStatementByUserAndStatuses Where securitySupport.getUserManager().getUser(userId).getAllGroups() throws an NPE, preventing the whole UI from displaying (see stack.txt). I suggest to wrap an optional around the call to get the UserManager. |
| Comments |
| Comment by Maxime Michel [ 13/May/20 ] |
|
Status update: Richard Unger's comment is no longer true. It is important in Magnolia 6.0+ that there is a user manager, even if to simply create on the fly users. This ticket is now about adding one to the module. |