-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73961/
-----------------------------------------------------------
(Updated April 29, 2022, 10:31 a.m.)
Review request for ranger, bhavik patel, Dhaval Shah, Kishor Gollapalliwar,
Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, Sailaja Polavarapu, and
Velmurugan Periasamy.
Bugs: RANGER-3735
https://issues.apache.org/jira/browse/RANGER-3735
Repository: ranger
Description
-------
There is a change in external user 'status' (i.e x_portal_user tables column)
which are getting synced into ranger admin, default ‘status’ value of synced
users are getting set as 0(disabled) which was not the case in ranger-2.1. This
is the behaviour change between 2.1 and later versions, Probably due this
(https://github.com/apache/ranger/commit/9be6a7d01f8d6a83e740ac27225eda1fff83501a).
- Added change to mark external users status as enable(1).
- written a java patch to update the status of existing external users.
Diffs (updated)
-----
security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql b852f43dd
security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql
3704ff598
security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql
be009d239
security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql
2f4db3e0e
security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql
f566955c6
security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java dbddef598
security-admin/src/main/java/org/apache/ranger/db/XXPortalUserDao.java
b4fdf027b
security-admin/src/main/java/org/apache/ranger/patch/PatchForExternalUserStatusUpdate_J10056.java
PRE-CREATION
security-admin/src/main/resources/META-INF/jpa_named_queries.xml b2a748f2c
Diff: https://reviews.apache.org/r/73961/diff/3/
Changes: https://reviews.apache.org/r/73961/diff/2-3/
Testing
-------
- All unit tests passed
- Verified fresh and upgrade(existing external users status are getting updated
to 1) scenario.
Thanks,
Mateen Mansoori
