> On Nov. 7, 2017, 9:09 a.m., Mehul Parikh wrote: > > kms/src/main/java/org/apache/hadoop/crypto/key/RangerKMSDB.java > > Line 198 (original), 185 (patched) > > <https://reviews.apache.org/r/63575/diff/1/?file=1881353#file1881353line198> > > > > @Zsombor : Can you please confirm if you have verified Ranger KMS > > sanity testing on SSL enabled MySQL DB ?
I've tested it with the command line tools, which are using RangerKMSDB directly on an SSL enabled MySQL and MariaDB. - Zsombor ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63575/#review190276 ----------------------------------------------------------- On Nov. 6, 2017, 10:22 a.m., Zsombor Gegesy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63575/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2017, 10:22 a.m.) > > > Review request for ranger. > > > Bugs: RANGER-1873 > https://issues.apache.org/jira/browse/RANGER-1873 > > > Repository: ranger > > > Description > ------- > > In RangerKMSDB, the updateDBSSLURL function is never called, because > getDBFlavor always return DB_FLAVOR_UNKNOWN, because it tries to read from > DB_PROPERTIES, instead of the configuration. The DB_PROPERTIES only contains > - at that point values for the keys: > PROPERTY_PREFIX+DB_DIALECT,PROPERTY_PREFIX+DB_DRIVER,PROPERTY_PREFIX+DB_URL. > > Fixed this issue, moved the DB_PROPERTIES static variable to an instance > variable, and simplified the string manipulation code a bit > > > Diffs > ----- > > kms/src/main/java/org/apache/hadoop/crypto/key/RangerKMSDB.java 649da30c5 > > > Diff: https://reviews.apache.org/r/63575/diff/1/ > > > Testing > ------- > > https://travis-ci.org/gzsombor/ranger/builds/297898874 > > > Thanks, > > Zsombor Gegesy > >
