----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72958/#review222065 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/security/context/RangerPreAuthSecurityHandler.java Line 96 (original), 97 (patched) <https://reviews.apache.org/r/72958/#comment311133> Please consider if this also needs to be changed. Also review other places in this file for the same. - Abhay Kulkarni On Oct. 13, 2020, 3:59 p.m., Dineshkumar Yadav wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72958/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2020, 3:59 p.m.) > > > Review request for ranger, Ankita Sinha, Gautam Borad, Kishor Gollapalliwar, > Abhay Kulkarni, Mehul Parikh, and Velmurugan Periasamy. > > > Repository: ranger > > > Description > ------- > > When import policy api is called and @PreAuthorize fails It does not return > proper response with status code. > > > Diffs > ----- > > > security-admin/src/main/java/org/apache/ranger/security/context/RangerPreAuthSecurityHandler.java > a4bf69486 > > > Diff: https://reviews.apache.org/r/72958/diff/1/ > > > Testing > ------- > > after changes it is returning response with status-code > > {"statusCode":401,"msgDesc":"User is not allowed to access the API"} > > > Thanks, > > Dineshkumar Yadav > >
