Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add upstream security information pass thru mechanism #1191

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

enicloom
Copy link
Contributor

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.

@enicloom enicloom requested a review from bhou2 September 22, 2023 20:44
@codecov
Copy link

codecov bot commented Sep 22, 2023

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%) ⬆️

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

Copy link
Contributor

@bhou2 bhou2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The code is clear and clean.

@enicloom enicloom merged commit c6c81dc into Netflix:master Sep 22, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants