-
Notifications
You must be signed in to change notification settings - Fork 621
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 GRPC proxy support #575
Conversation
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.
One question but otherwise LGTM. Are you running this in prod right now?
proxy/grpc_handler.go
Outdated
} | ||
|
||
newCtx := context.Background() | ||
conn, err := grpc.DialContext(newCtx, target.URL.Host, opts...) |
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.
Do you want to return conn
even if there was an error?
Thanks a lot @andyroyle for this. Are you running this in prod already? |
@magiconair yeah we have this in production now for about 3 weeks - it was a replacement for us using HaProxy and it’s integrated well with consul :) |
Thx Paul. |
Is this the same as #436? |
So this PR doesn't include those http/2 changes from #436 in, because it creates an entirely separate transport. It might still be worth merging that. |
Adds proper GRPC proxy support, including TLS upstream and TLS termination.