You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by st...@apache.org on 2022/06/19 11:52:29 UTC

[impala] 01/02: IMPALA-11371: Increase password complexity of users added in Ranger

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

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

commit f1291528c6a505453ab878920dc473bc35f14d7f
Author: Fang-Yu Rao <fa...@cloudera.com>
AuthorDate: Fri Jun 17 14:17:31 2022 -0700

    IMPALA-11371: Increase password complexity of users added in Ranger
    
    RANGER-3624 increased the minimum requirement for password complexity.
    This resulted in create-load-data.sh not being able to add users to
    Ranger, which are prerequisite for all the authorization-related tests.
    
    This patch increases the password complexity of the users to be added
    in Ranger so that those authorization-related tests could be run.
    
    Testing:
     - Verified locally that after this patch the users could be added in
       Ranger.
    
    Change-Id: Ib6265bbc123abcb572012015c71cb83e3fdfac26
    Reviewed-on: http://gerrit.cloudera.org:8080/18637
    Reviewed-by: Michael Smith <mi...@cloudera.com>
    Reviewed-by: Wenzhe Zhou <wz...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 testdata/cluster/ranger/setup/impala_user_non_owner.json.template | 2 +-
 testdata/cluster/ranger/setup/impala_user_owner.json.template     | 2 +-
 tests/authorization/test_ranger.py                                | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/testdata/cluster/ranger/setup/impala_user_non_owner.json.template b/testdata/cluster/ranger/setup/impala_user_non_owner.json.template
index 76f54b008..fad287ea6 100644
--- a/testdata/cluster/ranger/setup/impala_user_non_owner.json.template
+++ b/testdata/cluster/ranger/setup/impala_user_non_owner.json.template
@@ -1,6 +1,6 @@
 {
   "name" : "non_owner",
-  "password" : "password123",
+  "password" : "Password123",
   "userRoleList" : [ "ROLE_USER" ],
   "groupIdList" : [ "${GROUP_ID_NON_OWNER}" ]
 }
diff --git a/testdata/cluster/ranger/setup/impala_user_owner.json.template b/testdata/cluster/ranger/setup/impala_user_owner.json.template
index 36f2d3e95..bcb368e38 100644
--- a/testdata/cluster/ranger/setup/impala_user_owner.json.template
+++ b/testdata/cluster/ranger/setup/impala_user_owner.json.template
@@ -1,6 +1,6 @@
 {
   "name" : "${USER}",
-  "password" : "password123",
+  "password" : "Password123",
   "userRoleList" : [ "ROLE_USER" ],
   "groupIdList" : [ "${GROUP_ID_OWNER}" ]
 }
diff --git a/tests/authorization/test_ranger.py b/tests/authorization/test_ranger.py
index c538a6a16..e34f7b21b 100644
--- a/tests/authorization/test_ranger.py
+++ b/tests/authorization/test_ranger.py
@@ -655,7 +655,7 @@ class TestRanger(CustomClusterTestSuite):
     return json.loads(r.content)["policies"]
 
   def _add_ranger_user(self, user):
-    data = {"name": user, "password": "password123", "userRoleList": ["ROLE_USER"]}
+    data = {"name": user, "password": "Password123", "userRoleList": ["ROLE_USER"]}
     r = requests.post("{0}/service/xusers/secure/users".format(RANGER_HOST),
                       auth=RANGER_AUTH,
                       json=data, headers=REST_HEADERS)