feat: add --extra-header flag for custom gRPC headers#588
feat: add --extra-header flag for custom gRPC headers#588sputnikraket wants to merge 1 commit intoauthzed:mainfrom
Conversation
Signed-off-by: Erik Hennings <erik.hennings@freda.com>
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
I think this is fine, but what's the use case? I'm wondering if it's something that zed already supports. |
We're running SpiceDB behind a proxy that requires authentication (GCP Cloud Run), that can be specified either through the This setup is perhaps not ideal, but it works. |
tstirrat15
left a comment
There was a problem hiding this comment.
See question; otherwise LGTM
| md := metadata.New(headers) | ||
| ctx = metadata.NewOutgoingContext(ctx, md) |
There was a problem hiding this comment.
Does this logic functionally merge the headers, or would this overwrite headers if some other component added new headers?
Signed-off-by: Erik Hennings erik.hennings@freda.com
Description
This PR adds support for passing custom headers to gRPC requests via a new
--extra-headerflag.Testing
Manual testing
References