----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75330/#review227155 -----------------------------------------------------------
Ship it! Ship It! - Dineshkumar Yadav On Jan. 3, 2025, 12:40 p.m., Rakesh Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/75330/ > ----------------------------------------------------------- > > (Updated Jan. 3, 2025, 12:40 p.m.) > > > Review request for ranger, Dineshkumar Yadav, Kishor Gollapalliwar, Abhay > Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, > sanket shelar, Sailaja Polavarapu, and Velmurugan Periasamy. > > > Bugs: RANGER-5089 > https://issues.apache.org/jira/browse/RANGER-5089 > > > Repository: ranger > > > Description > ------- > > The API /service/xusers/secure/download/{invalid_service} returns a 204 No > Content status for invalid service names, instead of the expected 404 Not > Found. > > CURRENT BEHAVIOR: > The API /service/xusers/secure/download/{invalid_service} returns a 204 No > Content status even when the service name is invalid or does not exist. > > EXPECTED BEHAVIOR: > The API should return a 404 Not Found status when the provided service name > is invalid or does not exist. > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/rest/XUserREST.java > 9f7e79f7f > > > Diff: https://reviews.apache.org/r/75330/diff/1/ > > > Testing > ------- > > Tested and verified that the API > /service/xusers/secure/download/{invalid_service} correctly returns a 404 Not > Found status with an appropriate error message when an invalid or > non-existent service name is provided. > > > Thanks, > > Rakesh Gupta > >