Netflix / genie

Distributed Big Data Orchestration Service
https://netflix.github.io/genie
Apache License 2.0
1.71k stars 368 forks source link

add upstream security information pass thru mechanism #1191

Closed enicloom closed 1 year ago

enicloom commented 1 year ago

This pr adds a way for the clients that use the genie java-client to pass through customized security information.

Ideally, the security or authorization should be handled by the framework rather than app logic. However, due to the fact that the upstream clients of genie normally are some type of scheduler that runs job on user's behalf. The job authorization may need information beyond the client's own identity.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1191 (39ea16b) into master (fc1b554) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #1191   +/-   ##
=========================================
  Coverage     90.74%   90.74%           
- Complexity     3803     3809    +6     
=========================================
  Files           467      467           
  Lines         14963    14968    +5     
  Branches       1071     1071           
=========================================
+ Hits          13578    13583    +5     
  Misses          909      909           
  Partials        476      476           
Files Changed Coverage Δ
.../main/java/com/netflix/genie/client/JobClient.java 56.71% <100.00%> (+1.67%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more