----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74976/#review226424 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java Lines 200 (patched) <https://reviews.apache.org/r/74976/#comment314630> connection has not failed, rather you should ask use to provide a valid host as "localhost" or "127.0.0.1" is not allowed. - Pradeep Agrawal On May 8, 2024, 2:45 p.m., Rakesh Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74976/ > ----------------------------------------------------------- > > (Updated May 8, 2024, 2:45 p.m.) > > > Review request for ranger, Dineshkumar Yadav, Kishor Gollapalliwar, Mehul > Parikh, Pradeep Agrawal, and sanket shelar. > > > Bugs: RANGER-4782 > https://issues.apache.org/jira/browse/RANGER-4782 > > > Repository: ranger > > > Description > ------- > > Implement best coding practices for validating service configs > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java > b9d51602e > > security-admin/src/main/webapp/react-webapp/src/views/ServiceManager/TestConnection.jsx > 668de52e7 > > > Diff: https://reviews.apache.org/r/74976/diff/2/ > > > Testing > ------- > > Validated Service Connection failed, when using 'localhost' or '127.0.0.1' as > the hostname in the service configuration. > > Tested If a service configuration contains a null value, it won't be added to > the service. > > > Thanks, > > Rakesh Gupta > >
