Use lower-case gRPC header keys

[yang-g]

As upper-case letters are illegal per http2 spec.

[yang-g]

#2204

[tseaver]

@yang-g Thanks for the fix! Travis is going to barf whenever it gets around to running, because the unit tests assert the upper-case field names:

====================================================================== FAIL: test___call__ (gcloud.test__helpers.TestMetadataPlugin) ---------------------------------------------------------------------- Traceback (most recent call last): File "/home/tseaver/projects/agendaless/Google/src/gcloud-python/gcloud/test__helpers.py", line 845, in test___call__ self.assertEqual(callback_args, [(cb_headers, None)]) AssertionError: Lists differ: [([('authorization', 'Bearer F... != [([('Authorization', 'Bearer F... First differing element 0: ([('authorization', 'Bearer FOOBARBAZ'), ('user-agent', 'USER_AGENT')], None) ([('Authorization', 'Bearer FOOBARBAZ'), ('User-agent', 'USER_AGENT')], None) - [([('authorization', 'Bearer FOOBARBAZ'), ('user-agent', 'USER_AGENT')], None)] ? ^ ^ + [([('Authorization', 'Bearer FOOBARBAZ'), ('User-agent', 'USER_AGENT')], None)] ? ^ ^

Can you please update the PR accordingly?

@dhermes, @daspecster For future reference, here is the HTTP/2 spec reference which mandates lower-case for header field names (I was completely unaware of that requirement).

[tseaver]

I cancelled the not-yet-started Travis build after verifying that it would fail.

[yang-g]

@tseaver I pushed eebb83a about the same time of your comment. I did a search and did not find anything else. Could you restart the travis-ci or point me to more places to fix? Thanks.

[tseaver]

@yang-g I restarted the build. Sorry for my haste in cancelling -- Travis has been so glacially slow lately that we've been minimizing "known broken" jobs in the queue.

[dhermes]

@tseaver I think Travis have sorted out their issues

[tseaver]

@yang-g Thank you again!