[SECURITY-2] Email address usernames produce incorrect admincentral urls Created: 09/Nov/20  Updated: 19/Dec/23  Resolved: 15/Dec/23

Status: Closed
Project: Security
Component/s: None
Affects Version/s: 6.2.18
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Richard Gange Assignee: Unassigned
Resolution: Obsolete Votes: 1
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File jcr-browser.png     PNG File security-app.png    
Issue Links:
relation
Template:
Acceptance criteria:
Empty
Date of First Response:
Epic Link: Migrate security app to M6
Team: AdminX

 Description   

When using an email address for a public account the Security app shows an incorrect node path. Notice the difference between the Security app and the JCR Browser app.

Security app

/r/ri/richard.gange[commat]magnolia-cms.com

Jcr Browser

/public/r/ri/richard.gange@magnolia-cms.com

Notes
The side effect this issue is also producing (and should be tested) is when configuring the Find Bar to search this workspace you cannot reach the nodes.

Configure a search result supplier:
I created the following file: /my-module/searchResultSuppliers/security.yaml

workspace: users
nodeTypes:
  - mgnl:user
fullTextSearch: true
propertySearch: false
titleProperty: name
class: info.magnolia.periscope.search.jcr.JcrSearchResultSupplierDefinition
appName: security
subAppName: public
rootPath: /public
icon: icon-security-app

Try to search a user which has a username stored using email.

But we have a problem to open the right app and have the focus to the right line in the tree.

It works fine, if the name of the found node is a string like "<googleId>_google". The resulting URL is eg http://localhost:8080/dev/.magnolia/admincentral#app:security:public;/1/10/107241501216848955082_google:treeview:

It doesn't work for all users that are stored with their email addresses as name. The resulting URL is eg http://localhost:8080/dev/.magnolia/admincentral#app:security:public;/:treeview: (so the path is not added to the hash) the right url would be eg. http://localhost:8080/dev/.magnolia/admincentral#app:security:public;/e/ex/example[commat]mycompany.com:treeview:



 Comments   
Comment by Bertrand Renuart [ 09/Mar/23 ]

jsimak , rgange Can we close that ticket now?

Comment by Tom Wespi [ 15/Mar/23 ]

Bertrand Renuart Richard Gange Jaroslav Simak The bug is still here. It doesn't work for all users that are stored with their email addresses as name.

And as long the security app is not ported to UI6 this bug will probably stay forever...

Comment by Richard Gange [ 15/Dec/23 ]

Closing, The issue does not exist in the new app. The only problem now is you cannot create new accounts with email addresses. The @ is replaced with - now. Will create a follow up ticket about that.

Comment by Tom Wespi [ 19/Dec/23 ]

Not only that you cannot create accounts with @ in it, it also re-writes the current username of an existing account if you edit it.

Generated at Mon Feb 12 10:47:01 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.