-
Notifications
You must be signed in to change notification settings - Fork 218
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
feat(sdk/go): thread client token on RPC calls when provided #1389
Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #1389 +/- ##
==========================================
- Coverage 78.17% 77.98% -0.19%
==========================================
Files 43 43
Lines 3239 3239
==========================================
- Hits 2532 2526 -6
- Misses 565 569 +4
- Partials 142 144 +2 see 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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
* main: (36 commits) feat(sdk/go): thread client token on RPC calls when provided (#1389) docs: update .all-contributorsrc [skip ci] docs: update README.md [skip ci] chore(deps): bump google.golang.org/protobuf in /_tools (#1401) chore(deps): bump google.golang.org/protobuf from 1.29.0 to 1.29.1 (#1400) fix: return better error messages for grpc-gateway errors (#1397) fix: import for cockroach db (#1399) chore(deps): bump golang.org/x/tools from 0.6.0 to 0.7.0 in /_tools (#1394) chore: update readme w better usecase verbiage (#1395) fix(sdk/go): documentation and example corrections chore(deps): bump google.golang.org/protobuf from 1.28.1 to 1.29.0 (#1393) chore(deps): bump google.golang.org/protobuf in /_tools (#1391) chore(sdk/go): fix README markdown style fix(sdk/go): drop support for single field message mangling chore(sdk/go): add README chore(gowork): go edit -fmt feat(sdk/go): add doc.go feat(sdk/go): add static client token provider refactor(sdk/go): move grpc transport into its own subpackage chore(proto): regenerate RPC using protoc-gen-go-grpc v1.3.0 ...
* namespaces: (31 commits) Release/1.19 (#1403) feat(sdk/go): thread client token on RPC calls when provided (#1389) docs: update .all-contributorsrc [skip ci] docs: update README.md [skip ci] chore(deps): bump google.golang.org/protobuf in /_tools (#1401) chore(deps): bump google.golang.org/protobuf from 1.29.0 to 1.29.1 (#1400) fix: return better error messages for grpc-gateway errors (#1397) fix: import for cockroach db (#1399) chore(deps): bump golang.org/x/tools from 0.6.0 to 0.7.0 in /_tools (#1394) chore: update readme w better usecase verbiage (#1395) fix(sdk/go): documentation and example corrections chore(deps): bump google.golang.org/protobuf from 1.28.1 to 1.29.0 (#1393) chore(deps): bump google.golang.org/protobuf in /_tools (#1391) chore(sdk/go): fix README markdown style fix(sdk/go): drop support for single field message mangling chore(sdk/go): add README chore(gowork): go edit -fmt feat(sdk/go): add doc.go feat(sdk/go): add static client token provider refactor(sdk/go): move grpc transport into its own subpackage ...
Fixes FLI-253
This updates the generated
sdk/go
types to thread through client tokens acquired via theClientTokenProvider
interface onto gRPC outgoing metadata.This ensures that consumers only need to provide an implementation of the token provider and then their requests are authenticated.