When you are in the admin interface we need to check the validation. Admin shouldn't create an account with different domains. With this patch, we fix the problem.

CommentFileSizeAuthor
#5 check_user_register_admin-2909027-5.patch653 bytesrassoni
#3 check_user_register_admin-2909027-3.patch711 bytesrassoni
check_user_register_admin.patch688 bytesfastangel

Comments

fastangel created an issue. See original summary.

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Category: Bug report » Feature request
Status: Needs review » Needs work
Issue tags: +Needs tests

Thanks, I think this is useful but only if we add a bypass permission, see #2631058: Bypass restriction to role

rassoni’s picture

rassoni commented
Status: Needs work » Needs review
StatusFileSize
new check_user_register_admin-2909027-3.patch711 bytes

Added condition inside function

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Needs work

Still NW per #2

rassoni’s picture

rassoni commented
StatusFileSize
new check_user_register_admin-2909027-5.patch653 bytes

Bypass administrator permission as per #2

rassoni’s picture

rassoni commented
Status: Needs work » Needs review
pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Needs work

There's more than just adding a random string, please follow up on #2631058: Bypass restriction to role with the permission part