This is an automated email from the ASF dual-hosted git repository.

pradeep pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/master by this push:
     new afd54d6  RANGER-2400: policy name needs to be unique within security 
zone and service - part 4
afd54d6 is described below

commit afd54d6b88f4a34db1e63ca47caf28f48b9e8c77
Author: Abhay Kulkarni <[email protected]>
AuthorDate: Mon Apr 22 08:56:57 2019 -0700

    RANGER-2400: policy name needs to be unique within security zone and 
service - part 4
---
 security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql | 2 +-
 .../db/mysql/patches/037-create-security-zone-schema.sql           | 2 +-
 .../db/oracle/optimized/current/ranger_core_db_oracle.sql          | 2 +-
 .../db/oracle/patches/037-create-security-zone-schema.sql          | 2 +-
 .../db/postgres/optimized/current/ranger_core_db_postgres.sql      | 2 +-
 .../db/postgres/patches/037-create-security-zone-schema.sql        | 2 +-
 .../sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql   | 2 +-
 .../db/sqlanywhere/patches/037-create-security-zone-schema.sql     | 4 ++--
 .../db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql    | 2 +-
 .../db/sqlserver/patches/037-create-security-zone-schema.sql       | 2 +-
 .../org/apache/ranger/service/RangerSecurityZoneServiceBase.java   | 7 ++++++-
 11 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql 
b/security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql
index 63e1b6f..769afb5 100644
--- a/security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql
+++ b/security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql
@@ -1519,7 +1519,7 @@ INSERT INTO 
x_user(create_time,update_time,added_by_id,upd_by_id,user_name,descr
 
 INSERT INTO `x_modules_master` 
(`create_time`,`update_time`,`added_by_id`,`upd_by_id`,`module`,`url`) VALUES 
(UTC_TIMESTAMP(),UTC_TIMESTAMP(),getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Resource
 Based 
Policies',''),(UTC_TIMESTAMP(),UTC_TIMESTAMP(),getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Users/Groups',''),(UTC_TIMESTAMP(),UTC_TIMESTAMP(),getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Reports',''),(UTC_TIMESTAMP(),UTC_TIMESTAM
 [...]
 INSERT INTO `x_modules_master` 
(`create_time`,`update_time`,`added_by_id`,`upd_by_id`,`module`,`url`) VALUES 
(UTC_TIMESTAMP(),UTC_TIMESTAMP(),getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Security
 Zone','');
-INSERT INTO x_security_zone(id, create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES (1, 
UTC_TIMESTAMP(),UTC_TIMESTAMP(), getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, '', '', 'Unzoned zone');
+INSERT INTO x_security_zone(id, create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES (1, 
UTC_TIMESTAMP(),UTC_TIMESTAMP(), getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, ' ', '', 'Unzoned zone');
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('CORE_DB_SCHEMA',UTC_TIMESTAMP(),'Ranger 
1.0.0',UTC_TIMESTAMP(),'localhost','Y');
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('001',UTC_TIMESTAMP(),'Ranger 1.0.0',UTC_TIMESTAMP(),'localhost','Y');
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('002',UTC_TIMESTAMP(),'Ranger 1.0.0',UTC_TIMESTAMP(),'localhost','Y');
diff --git 
a/security-admin/db/mysql/patches/037-create-security-zone-schema.sql 
b/security-admin/db/mysql/patches/037-create-security-zone-schema.sql
index b6c1080..8c9e073 100644
--- a/security-admin/db/mysql/patches/037-create-security-zone-schema.sql
+++ b/security-admin/db/mysql/patches/037-create-security-zone-schema.sql
@@ -202,7 +202,7 @@ drop procedure if exists add_unzone_entry;
 delimiter ;;
 create procedure add_unzone_entry() begin
 if not exists (select * from x_security_zone where id=1 and name=' ') then
-       INSERT INTO x_security_zone(id, create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES (1, 
UTC_TIMESTAMP(),UTC_TIMESTAMP(), getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, '', '', 'Unzoned zone');
+       INSERT INTO x_security_zone(id, create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES (1, 
UTC_TIMESTAMP(),UTC_TIMESTAMP(), getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, ' ', '', 'Unzoned zone');
 end if;
 end;;
 delimiter ;
diff --git 
a/security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql 
b/security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql
index 8cb1ba1..9a9e36b 100644
--- a/security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql
+++ b/security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql
@@ -1693,7 +1693,7 @@ INSERT INTO x_modules_master 
VALUES(X_MODULES_MASTER_SEQ.NEXTVAL,sys_extract_utc
 INSERT INTO x_modules_master 
VALUES(X_MODULES_MASTER_SEQ.NEXTVAL,sys_extract_utc(systimestamp),sys_extract_utc(systimestamp),getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Key
 Manager','');
 INSERT INTO x_modules_master 
VALUES(X_MODULES_MASTER_SEQ.NEXTVAL,sys_extract_utc(systimestamp),sys_extract_utc(systimestamp),getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Tag
 Based Policies','');
 INSERT INTO x_modules_master 
VALUES(X_MODULES_MASTER_SEQ.NEXTVAL,sys_extract_utc(systimestamp),sys_extract_utc(systimestamp),getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Security
 Zone','');
-INSERT INTO x_security_zone(id, create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES 
(X_SECURITY_ZONE_SEQ.NEXTVAL, sys_extract_utc(systimestamp), 
sys_extract_utc(systimestamp), getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, '', '','Unzoned zone');
+INSERT INTO x_security_zone(id, create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES 
(X_SECURITY_ZONE_SEQ.NEXTVAL, sys_extract_utc(systimestamp), 
sys_extract_utc(systimestamp), getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, ' ', '','Unzoned zone');
 commit;
 INSERT INTO x_db_version_h 
(id,version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
(X_DB_VERSION_H_SEQ.nextval, 
'CORE_DB_SCHEMA',sys_extract_utc(systimestamp),'Ranger 
1.0.0',sys_extract_utc(systimestamp),'localhost','Y');
 INSERT INTO x_db_version_h 
(id,version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
(X_DB_VERSION_H_SEQ.nextval, '001',sys_extract_utc(systimestamp),'Ranger 
1.0.0',sys_extract_utc(systimestamp),'localhost','Y');
diff --git 
a/security-admin/db/oracle/patches/037-create-security-zone-schema.sql 
b/security-admin/db/oracle/patches/037-create-security-zone-schema.sql
index efb51e2..8ed2b66 100644
--- a/security-admin/db/oracle/patches/037-create-security-zone-schema.sql
+++ b/security-admin/db/oracle/patches/037-create-security-zone-schema.sql
@@ -231,7 +231,7 @@ DECLARE
 BEGIN
 Select count(*) into v_column_exists from x_security_zone where id = 1 and 
name = ' ';
        if (v_column_exists = 0) then
-               INSERT INTO x_security_zone(id, create_time, update_time, 
added_by_id, upd_by_id, version, name, jsonData, description) VALUES 
(X_SECURITY_ZONE_SEQ.NEXTVAL, sys_extract_utc(systimestamp), 
sys_extract_utc(systimestamp), getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, '', '','Unzoned zone');
+               INSERT INTO x_security_zone(id, create_time, update_time, 
added_by_id, upd_by_id, version, name, jsonData, description) VALUES 
(X_SECURITY_ZONE_SEQ.NEXTVAL, sys_extract_utc(systimestamp), 
sys_extract_utc(systimestamp), getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, ' ', '','Unzoned zone');
                commit;
        end if;
 end;/
diff --git 
a/security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql 
b/security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql
index fa768cc..df4201d 100644
--- a/security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql
+++ b/security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql
@@ -1628,7 +1628,7 @@ INSERT INTO 
x_user(CREATE_TIME,UPDATE_TIME,user_name,status,descr)VALUES(current
 
 INSERT INTO 
x_modules_master(create_time,update_time,added_by_id,upd_by_id,module,url) 
VALUES(current_timestamp,current_timestamp,getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Tag
 Based Policies','');
 INSERT INTO 
x_modules_master(create_time,update_time,added_by_id,upd_by_id,module,url) 
VALUES(current_timestamp,current_timestamp,getXportalUIdByLoginId('admin'),getXportalUIdByLoginId('admin'),'Security
 Zone','');
-INSERT INTO x_security_zone(create_time, update_time, added_by_id, upd_by_id, 
version, name, jsonData, description) VALUES (current_timestamp, 
current_timestamp, getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, '', '', 'Unzoned zone');
+INSERT INTO x_security_zone(create_time, update_time, added_by_id, upd_by_id, 
version, name, jsonData, description) VALUES (current_timestamp, 
current_timestamp, getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, ' ', '', 'Unzoned zone');
 
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('CORE_DB_SCHEMA',current_timestamp,'Ranger 
1.0.0',current_timestamp,'localhost','Y');
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('016',current_timestamp,'Ranger 1.0.0',current_timestamp,'localhost','Y');
diff --git 
a/security-admin/db/postgres/patches/037-create-security-zone-schema.sql 
b/security-admin/db/postgres/patches/037-create-security-zone-schema.sql
index 7125118..f55e5b1 100644
--- a/security-admin/db/postgres/patches/037-create-security-zone-schema.sql
+++ b/security-admin/db/postgres/patches/037-create-security-zone-schema.sql
@@ -239,7 +239,7 @@ DECLARE
 BEGIN
   select count(*) into v_column_exists from x_security_zone where id=1 and 
name=' ';
    IF v_column_exists = 0 THEN
-               INSERT INTO x_security_zone(create_time, update_time, 
added_by_id, upd_by_id, version, name, jsonData, description) VALUES 
(current_timestamp, current_timestamp, getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, '', '', 'Unzoned zone');
+               INSERT INTO x_security_zone(create_time, update_time, 
added_by_id, upd_by_id, version, name, jsonData, description) VALUES 
(current_timestamp, current_timestamp, getXportalUIdByLoginId('admin'), 
getXportalUIdByLoginId('admin'), 1, ' ', '', 'Unzoned zone');
        END IF;
 END;
 $$ LANGUAGE plpgsql;
diff --git 
a/security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql
 
b/security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql
index 0f877b5..a2d4137 100644
--- 
a/security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql
+++ 
b/security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql
@@ -1949,7 +1949,7 @@ INSERT INTO 
x_portal_user_role(create_time,update_time,added_by_id,upd_by_id,use
 GO
 INSERT INTO 
x_user(create_time,update_time,added_by_id,upd_by_id,user_name,descr,status) 
values (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP,NULL,NULL,'rangertagsync','rangertagsync',0);
 GO
-INSERT INTO x_security_zone(create_time, update_time, added_by_id, upd_by_id, 
version, name, jsonData, description) VALUES (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP, dbo.getXportalUIdByLoginId('admin'), 
dbo.getXportalUIdByLoginId('admin'), 1, '', '', 'Unzoned zone');
+INSERT INTO x_security_zone(create_time, update_time, added_by_id, upd_by_id, 
version, name, jsonData, description) VALUES (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP, dbo.getXportalUIdByLoginId('admin'), 
dbo.getXportalUIdByLoginId('admin'), 1, ' ', '', 'Unzoned zone');
 GO
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('CORE_DB_SCHEMA',CURRENT_TIMESTAMP,'Ranger 
1.0.0',CURRENT_TIMESTAMP,'localhost','Y');
 GO
diff --git 
a/security-admin/db/sqlanywhere/patches/037-create-security-zone-schema.sql 
b/security-admin/db/sqlanywhere/patches/037-create-security-zone-schema.sql
index 9c66c53..ce3f342 100644
--- a/security-admin/db/sqlanywhere/patches/037-create-security-zone-schema.sql
+++ b/security-admin/db/sqlanywhere/patches/037-create-security-zone-schema.sql
@@ -72,7 +72,7 @@ ALTER TABLE dbo.x_security_zone ADD CONSTRAINT 
x_security_zone_FK_added_by_id FO
 GO
 ALTER TABLE dbo.x_security_zone ADD CONSTRAINT x_security_zone_FK_upd_by_id 
FOREIGN KEY(upd_by_id) REFERENCES dbo.x_portal_user (id)
 GO
-INSERT INTO x_security_zone(create_time, update_time, added_by_id, upd_by_id, 
version, name, jsonData, description) VALUES (NULL, NULL, 1, 1, 1, "", "", 
"Unzoned zone");
+INSERT INTO x_security_zone(create_time, update_time, added_by_id, upd_by_id, 
version, name, jsonData, description) VALUES (NULL, NULL, 1, 1, 1, ' ', '', 
'Unzoned zone');
 GO
 CREATE TABLE dbo.x_ranger_global_state(
        id bigint IDENTITY NOT NULL,
@@ -223,7 +223,7 @@ BEGIN
 END;
 GO
 IF NOT EXISTS(select * from x_security_zone where id = 1 and name=' ') THEN
-       INSERT INTO x_security_zone(create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP, dbo.getXportalUIdByLoginId('admin'), 
dbo.getXportalUIdByLoginId('admin'), 1, '', '', 'Unzoned zone');
+       INSERT INTO x_security_zone(create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP, dbo.getXportalUIdByLoginId('admin'), 
dbo.getXportalUIdByLoginId('admin'), 1, ' ', '', 'Unzoned zone');
 END IF;
 GO
 IF NOT EXISTS(select * from SYS.SYSCOLUMNS where tname = 'x_policy' and 
cname='zone_id') THEN
diff --git 
a/security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql 
b/security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql
index 44c1722..1f3ccbf 100644
--- a/security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql
+++ b/security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql
@@ -3656,7 +3656,7 @@ insert into x_user 
(CREATE_TIME,UPDATE_TIME,user_name,status,descr) values (CURR
 insert into x_portal_user 
(CREATE_TIME,UPDATE_TIME,FIRST_NAME,LAST_NAME,PUB_SCR_NAME,LOGIN_ID,PASSWORD,EMAIL,STATUS)
 values 
(CURRENT_TIMESTAMP,CURRENT_TIMESTAMP,'rangertagsync','','rangertagsync','rangertagsync','f5820e1229418dcf2575908f2c493da5','rangertagsync',1);
 insert into x_portal_user_role 
(CREATE_TIME,UPDATE_TIME,USER_ID,USER_ROLE,STATUS) values 
(CURRENT_TIMESTAMP,CURRENT_TIMESTAMP,dbo.getXportalUIdByLoginId('rangertagsync'),'ROLE_SYS_ADMIN',1);
 insert into x_user (CREATE_TIME,UPDATE_TIME,user_name,status,descr) values 
(CURRENT_TIMESTAMP,CURRENT_TIMESTAMP,'rangertagsync',0,'rangertagsync');
-INSERT INTO x_security_zone(create_time, update_time, added_by_id, upd_by_id, 
version, name, jsonData, description) VALUES (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP, dbo.getXportalUIdByLoginId('admin'), 
dbo.getXportalUIdByLoginId('admin'), 1, '', '', 'Unzoned zone');
+INSERT INTO x_security_zone(create_time, update_time, added_by_id, upd_by_id, 
version, name, jsonData, description) VALUES (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP, dbo.getXportalUIdByLoginId('admin'), 
dbo.getXportalUIdByLoginId('admin'), 1, ' ', '', 'Unzoned zone');
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('CORE_DB_SCHEMA',CURRENT_TIMESTAMP,'Ranger 
1.0.0',CURRENT_TIMESTAMP,'localhost','Y');
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('016',CURRENT_TIMESTAMP,'Ranger 1.0.0',CURRENT_TIMESTAMP,'localhost','Y');
 INSERT INTO x_db_version_h 
(version,inst_at,inst_by,updated_at,updated_by,active) VALUES 
('018',CURRENT_TIMESTAMP,'Ranger 1.0.0',CURRENT_TIMESTAMP,'localhost','Y');
diff --git 
a/security-admin/db/sqlserver/patches/037-create-security-zone-schema.sql 
b/security-admin/db/sqlserver/patches/037-create-security-zone-schema.sql
index 8156471..e5d558f 100644
--- a/security-admin/db/sqlserver/patches/037-create-security-zone-schema.sql
+++ b/security-admin/db/sqlserver/patches/037-create-security-zone-schema.sql
@@ -437,7 +437,7 @@ END
 GO
 IF NOT EXISTS(select * from x_security_zone where id = 1 and name=' ')
 BEGIN
-       INSERT INTO x_security_zone(create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP, dbo.getXportalUIdByLoginId('admin'), 
dbo.getXportalUIdByLoginId('admin'), 1, '', '', 'Unzoned zone');
+       INSERT INTO x_security_zone(create_time, update_time, added_by_id, 
upd_by_id, version, name, jsonData, description) VALUES (CURRENT_TIMESTAMP, 
CURRENT_TIMESTAMP, dbo.getXportalUIdByLoginId('admin'), 
dbo.getXportalUIdByLoginId('admin'), 1, ' ', '', 'Unzoned zone');
 END
 GO
 IF NOT EXISTS(select * from INFORMATION_SCHEMA.columns where table_name = 
'x_policy' and column_name in('zone_id'))
diff --git 
a/security-admin/src/main/java/org/apache/ranger/service/RangerSecurityZoneServiceBase.java
 
b/security-admin/src/main/java/org/apache/ranger/service/RangerSecurityZoneServiceBase.java
index 87e5c61..586a6b7 100644
--- 
a/security-admin/src/main/java/org/apache/ranger/service/RangerSecurityZoneServiceBase.java
+++ 
b/security-admin/src/main/java/org/apache/ranger/service/RangerSecurityZoneServiceBase.java
@@ -17,6 +17,7 @@
 
 package org.apache.ranger.service;
 
+import org.apache.commons.lang.StringUtils;
 import org.apache.ranger.common.SearchField;
 import org.apache.ranger.common.SortField;
 import org.apache.ranger.entity.XXSecurityZone;
@@ -45,7 +46,11 @@ public abstract class RangerSecurityZoneServiceBase<T 
extends XXSecurityZone, V
 
        @Override
        protected V mapEntityToViewBean(V vObj, T xObj) {
-               vObj.setName(xObj.getName());
+               if 
(xObj.getId().equals(RangerSecurityZone.RANGER_UNZONED_SECURITY_ZONE_ID)) {
+                       vObj.setName(StringUtils.EMPTY);
+               } else {
+                       vObj.setName(xObj.getName());
+               }
                vObj.setDescription(xObj.getDescription());
                return vObj;
        }

Reply via email to