ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dhaval Shah <dhavalshah9...@gmail.com>
Subject Re: Review Request 72847: RANGER-2998 : API for Ranger KMS service status
Date Wed, 16 Sep 2020 18:33:35 GMT


> On Sept. 16, 2020, 3:44 p.m., Ramesh Mani wrote:
> > kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMDCFilter.java
> > Lines 77 (patched)
> > <https://reviews.apache.org/r/72847/diff/1/?file=2239325#file2239325line78>
> >
> >     do we need to check only the "/kms/api/status" path here and rest go through
the regular flow?

Yes. APIs such as "/kms/v1/" are written in KMS.java require authentication and authorization.


- Dhaval


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72847/#review221875
-----------------------------------------------------------


On Sept. 16, 2020, 6:31 p.m., Dhaval Shah wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72847/
> -----------------------------------------------------------
> 
> (Updated Sept. 16, 2020, 6:31 p.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Dineshkumar Yadav, Jayendra Parab, Kishor Gollapalliwar,
Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2998
>     https://issues.apache.org/jira/browse/RANGER-2998
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Need to construct the API which can be accessible without user credentials for Ranger
KMS service status. So when service Ranger KMS is successfully started there should be status
saying "Ranger KMS is up and running" when we hit particular API.
> 
> 
> Diffs
> -----
> 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAuthenticationFilter.java
944b3d483 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMDCFilter.java da8f71599

>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/RangerKMSRestApi.java PRE-CREATION

> 
> 
> Diff: https://reviews.apache.org/r/72847/diff/1/
> 
> 
> Testing
> -------
> 
> Successfully tested the status API.
> 
> 
> Thanks,
> 
> Dhaval Shah
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message