[MGNLUI-2793] Wrong UserManager is used during creation of new user Created: 26/Feb/14 Updated: 06/May/14 Resolved: 23/Mar/14 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | security app |
| Affects Version/s: | None |
| Fix Version/s: | 5.2.4 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Milan Divilek | Assignee: | Milan Divilek |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| 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)
|
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
| Description |
|
info.magnolia.security.app.dialog.action.SaveUserDialogAction for creation of new user everytime uses first usermanager which support creation of user, but instead of it UserManager for appropriate realm should be used. Problem is in method SaveUserDialogAction#createOrUpdateUser where UserManager is obtained by securitySupport.getUserManager(). It returns info.magnolia.cms.security.DelegatingUserManager and DelegatingUserManager.createUser method uses first UserManager which support creation of the user. You can't see any problems with standard magnolia installation. Because standard setting of userManagers look like this:
- server
-- security
--- userManagers
---- system
---- admin
---- public
So when you are creating admin user by using Security app. Then userManager for system realm is used and #createUser method in system user manager can handle it. But problem with this starts occur when you are customize userManagers setting. If you add your custom userManger(which has custom #createUser method) to first place in userManagers then issue will pop-up. |
| Comments |
| Comment by Milan Divilek [ 09/Apr/14 ] |
|
On 5.2.x branch committed under old ticket number |