----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73961/#review224381 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/patch/PatchForExternalUserStatusUpdate_J10056.java Lines 22 (patched) <https://reviews.apache.org/r/73961/#comment313217> remove the log4j logger. - bhavik patel On April 27, 2022, 3:02 p.m., Mateen Mansoori wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73961/ > ----------------------------------------------------------- > > (Updated April 27, 2022, 3:02 p.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 > ----- > > 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/1/ > > > Testing > ------- > > - All unit tests passed > - Verified fresh and upgrade(existing external users status are getting > updated to 1) scenario. > > > Thanks, > > Mateen Mansoori > >