----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44984/#review125761 -----------------------------------------------------------
I reviewed the code from a crypto point of view with Abe. It meets all the crypto best practices that I'm aware of. +1 from that aspect. - Mike Yoder On March 28, 2016, 9:52 p.m., Abraham Fine wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44984/ > ----------------------------------------------------------- > > (Updated March 28, 2016, 9:52 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2889 > https://issues.apache.org/jira/browse/SQOOP-2889 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > provide a mechanism to encrypt sensitive information in the repository > > > Diffs > ----- > > > common-test/src/main/java/org/apache/sqoop/common/test/db/DerbyProvider.java > 839e5618e3cd86a1df24b53b04aae3721b56ed27 > common/src/main/java/org/apache/sqoop/error/code/CommonRepositoryError.java > 37eb04aaa30f24577ff755e361e6d0a9d47be37c > common/src/main/java/org/apache/sqoop/error/code/PasswordError.java > PRE-CREATION > common/src/main/java/org/apache/sqoop/utils/PasswordUtils.java PRE-CREATION > common/src/main/java/org/apache/sqoop/utils/ProcessUtils.java > 0a32d6866dd348c598e8445f424c467e2781e0cf > common/src/test/java/org/apache/sqoop/utils/TestPasswordUtils.java > PRE-CREATION > core/src/main/java/org/apache/sqoop/core/SqoopServer.java > 80a7b88ac4704972f1de0ddaf21869feedba4aa8 > core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java > 5b70f951c98dec7d20ff446f52418234fd98e454 > core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryHandler.java > feab2ad8360b7d978b263dc0da1d2ff2578f0555 > core/src/main/java/org/apache/sqoop/repository/MasterKeyManager.java > PRE-CREATION > core/src/main/java/org/apache/sqoop/repository/Repository.java > 03989a3c053edfc51364e9fb368ec970c1dfecee > core/src/main/java/org/apache/sqoop/security/SecurityConstants.java > 0241c8692fef65866160e0bec53cc5c2da1c17e5 > core/src/main/java/org/apache/sqoop/security/SecurityError.java > 988e425a041a5829ad95508a1e9b20d0992c868e > core/src/test/java/org/apache/sqoop/repository/TestMasterKeyManager.java > PRE-CREATION > dist/src/main/conf/sqoop.properties > 767d3f2b3ddf8ca978830e37a321d9defbafc57d > > repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java > 15cc41b83adc59c266cb899a19edfcc5fc2ee343 > > repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryInsertUpdateDeleteSelectQuery.java > ae16b8517daf4d40fd2ebf4a473e16e9083740d6 > > repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositorySchemaConstants.java > d1940e82c02039c144c58a80b3da07f6d2e38b27 > > repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java > ee5e8d10f16365fa55574a819501b6ea48ffe30a > > repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaCreateQuery.java > 177003671863244a094abdb6f4a0184d38e79823 > > repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaUpgradeQuery.java > 5081b82ae2cbbd06cef9061b333dd4ec0b61b815 > > repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/DerbyTestCase.java > e4cca07fd062a64ac161929943dbd70b23a01c7f > > repository/repository-mysql/src/main/java/org/apache/sqoop/repository/mysql/MySqlRepositoryHandler.java > 2c74c323bf64f5c4b120aa745b1e7208e4ddfb3d > > repository/repository-mysql/src/main/java/org/apache/sqoop/repository/mysql/MySqlSchemaCreateQuery.java > 47f12fe457bc84fc36444e9aeb0b5fbc5a16356f > > repository/repository-mysql/src/main/java/org/apache/sqoop/repository/mysql/MysqlRepositoryInsertUpdateDeleteSelectQuery.java > 4c295c017195bd3b5ee79a6961814f3d8c8f9e17 > > repository/repository-postgresql/src/main/java/org/apache/sqoop/repository/postgresql/PostgresqlRepositoryHandler.java > 400d706c76bbb3e45f535556e925d1debf557928 > > repository/repository-postgresql/src/main/java/org/apache/sqoop/repository/postgresql/PostgresqlSchemaCreateQuery.java > 8358df0d90ef31fad02cf48df198f5c263a3915e > > repository/repository-postgresql/src/main/java/org/apache/sqoop/repository/postgresql/PostgresqlSchemaUpgradeQuery.java > 52954e6946772d4cff268d88aa0bfbc4a9d11aa9 > server/src/main/java/org/apache/sqoop/server/SqoopJettyServer.java > 00aa214822a660b2d623812d4a7469f5855b56b6 > shell/src/main/java/org/apache/sqoop/shell/SetTruststoreFunction.java > 0b603a1d2233746110cf6e21bd869b930a903a65 > test/src/main/java/org/apache/sqoop/test/minicluster/SqoopMiniCluster.java > c7a4db871c2e147f81455585aaeef71660c45922 > > Diff: https://reviews.apache.org/r/44984/diff/ > > > Testing > ------- > > > Thanks, > > Abraham Fine > >
