-
-
Notifications
You must be signed in to change notification settings - Fork 148
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 basic behavior of authorization webhook #188
Conversation
Codecov Report
@@ Coverage Diff @@
## main #188 +/- ##
==========================================
+ Coverage 61.45% 61.68% +0.22%
==========================================
Files 39 40 +1
Lines 3386 3440 +54
==========================================
+ Hits 2081 2122 +41
- Misses 1117 1124 +7
- Partials 188 194 +6
Continue to review full report at Codecov.
|
f23d9aa
to
a515688
Compare
The following additional work will be done in the follow-up PR.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
What this PR does / why we need it:
Add authorization webhook. In this PR, we only implement the default behavior.
RequestHeader
because we can carry the version in gRPC metadata.Which issue(s) this PR fixes:
Fixes #182
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation:
Checklist: