Yair Zaslavsky has uploaded a new change for review.

Change subject: aaa: Fixing missing name field for admin user
......................................................................

aaa: Fixing missing name field for admin user

Topic: AAA
Change-Id: Ie4ae5e103ecc9f85fbeaef6c9bc3351e9d4beb21
Bug-url: https://bugzilla.redhat.com/1101147
Signed-off-by: Yair Zaslavsky <yzasl...@redhat.com>
---
M 
backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/internal/InternalAuthz.java
A packaging/dbscripts/upgrade/03_06_0030_set_name_for_internal_user.sql
2 files changed, 6 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/88/30488/1

diff --git 
a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/internal/InternalAuthz.java
 
b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/internal/InternalAuthz.java
index fb2c420..44f33d1 100644
--- 
a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/internal/InternalAuthz.java
+++ 
b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/internal/InternalAuthz.java
@@ -112,12 +112,16 @@
                         Authz.ContextKeys.AVAILABLE_NAMESPACES,
                         Arrays.asList(NAMESPACE)
                         );
+        String userName = configuration.getProperty("config.authz.user.name");
         adminUser = new ExtMap().mput(
                 Authz.PrincipalRecord.NAMESPACE,
                 NAMESPACE
                 ).mput(
                         Authz.PrincipalRecord.NAME,
-                        configuration.getProperty("config.authz.user.name")
+                        userName
+                ).mput(
+                        Authz.PrincipalRecord.FIRST_NAME,
+                        userName
                 ).mput(
                         Authz.PrincipalRecord.ID,
                         configuration.getProperty("config.authz.user.id")
diff --git 
a/packaging/dbscripts/upgrade/03_06_0030_set_name_for_internal_user.sql 
b/packaging/dbscripts/upgrade/03_06_0030_set_name_for_internal_user.sql
new file mode 100644
index 0000000..2ac8652
--- /dev/null
+++ b/packaging/dbscripts/upgrade/03_06_0030_set_name_for_internal_user.sql
@@ -0,0 +1 @@
+update users set name = username where domain='internal';


-- 
To view, visit http://gerrit.ovirt.org/30488
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie4ae5e103ecc9f85fbeaef6c9bc3351e9d4beb21
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to