-
Notifications
You must be signed in to change notification settings - Fork 863
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
Enable token authorization and model control for gRPC #3238
Enable token authorization and model control for gRPC #3238
Conversation
bb53231
to
71c8371
Compare
a2a9ed9
to
bea375d
Compare
Why is gRPC unable to use the original token authorization handler? |
The token authorization implementation is protocol agnostic and can be shared by both the HTTP and gRPC handlers. Since |
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.
Passing the pytests so 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
Description
Enable and test token authorization, model control and allowed_urls for gRPC
Type of change
Feature/Issue validation/testing
test_gRPC_token_authorization.py
test_gRPC_model_control.py
test_gRPC_allowed_urls.py