remove headers that make grpc upset

googleapis · Jun 25, 2019 · 282fa8c · 282fa8c
1 parent df9d83f
commit 282fa8c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/server/util.go b/server/util.go
@@ -79,6 +79,11 @@ func prepareHeaders(ctx context.Context, hdr http.Header) context.Context {
 	// delete application/x-protobuf content-type
 	hdr.Del("content-type")
 
+	// delete headers that make gRPC upset
+	hdr.Del("user-agent")
+	hdr.Del("content-length")
+	hdr.Del("accept-encoding")
+
 	// keep every other headerin case-sensitive
 	// way, important for copying e.g. Authorization
 	for key, val := range hdr {

diff --git a/server/util_test.go b/server/util_test.go
@@ -53,6 +53,9 @@ func Test_prepareHeaders(t *testing.T) {
 	parent := context.Background()
 	hdr := make(http.Header)
 	hdr.Add("Content-Type", "test")
+	hdr.Add("accept-encoding", "blah")
+	hdr.Add("content-length", "7")
+	hdr.Add("user-agent", "whoever")
 	hdr.Add("Foo", "bar")
 
 	want := map[string][]string{