tree 3c6a8af0fd5fdc04cfea418925684dbd46091a1c
parent 473217c7f590f56568f04c71c91866d794beb596
author Hallgrimur Gunnarsson <halg@google.com> 1573072889 +0000
committer Chris Broadfoot <cbro@google.com> 1573156300 +0000

transport/http: fix the setting of custom HTTP headers

This fixes a bug where the custom X-Goog headers (like request reason)
are only set if the User-Agent header is also set. Looking at the commit
history, it seems that User-Agent was the only custom header at one point
and the code would return early (in dial.go:112) if it was empty as an
optimization. This logic was not updated when more custom headers were
added, which introduced a bug.

Change-Id: I9b46655628bc682eddede14f02dcc871fec04bc6
Reviewed-on: https://code-review.googlesource.com/c/google-api-go-client/+/47932
Reviewed-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Shin Fan <shinfan@google.com>
Reviewed-by: Chris Broadfoot <cbro@google.com>