-
Notifications
You must be signed in to change notification settings - Fork 10
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
Issue #228: Fix authorization in gRPC Calls #229
Conversation
Codecov Report
@@ Coverage Diff @@
## master #229 +/- ##
==========================================
+ Coverage 41.82% 42.07% +0.24%
==========================================
Files 98 99 +1
Lines 4368 4409 +41
==========================================
+ Hits 1827 1855 +28
- Misses 2385 2398 +13
Partials 156 156
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
--Wrote a custom auth interceptor which calls custom AuthFuncOverride with request body --Custom version of AuthFuncOverride accepting request body --Modified implementations of AuthFuncOverride in admin, publisher and subscriber server to accept request body --New file: metro/service/web/server.go having the implementation of fetching the project id from request payload --Fetch project-id from request payload using resources' name and extracted using regex capturing group
--Reverted code commented during testing
--Fixed breaking UTs
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
UnaryServerAuthInterceptor
andserviceAuthFuncOverride
interface which declares theAuthFuncOverride
function. If the server (read controller) implements the interface function, the override function is called with the request body and the full function name.grpc_auth.UnaryServerInterceptor
, except that the override function gets the request body.