{"action":"created","issue":{"url":"https://api.github.com/repos/grpc/grpc/issues/21196","repository_url":"https://api.github.com/repos/grpc/grpc","labels_url":"https://api.github.com/repos/grpc/grpc/issues/21196/labels{/name}","comments_url":"https://api.github.com/repos/grpc/grpc/issues/21196/comments","events_url":"https://api.github.com/repos/grpc/grpc/issues/21196/events","html_url":"https://github.com/grpc/grpc/pull/21196","id":522546595,"node_id":"MDExOlB1bGxSZXF1ZXN0MzQwNzI3NzQ1","number":21196,"title":"[Exposing ALTS Context 2/2] Provide a Utility Function to Get Context","user":{"login":"ZhenLian","id":23140989,"node_id":"MDQ6VXNlcjIzMTQwOTg5","avatar_url":"https://avatars3.githubusercontent.com/u/23140989?v=4","gravatar_id":"","url":"https://api.github.com/users/ZhenLian","html_url":"https://github.com/ZhenLian","followers_url":"https://api.github.com/users/ZhenLian/followers","following_url":"https://api.github.com/users/ZhenLian/following{/other_user}","gists_url":"https://api.github.com/users/ZhenLian/gists{/gist_id}","starred_url":"https://api.github.com/users/ZhenLian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ZhenLian/subscriptions","organizations_url":"https://api.github.com/users/ZhenLian/orgs","repos_url":"https://api.github.com/users/ZhenLian/repos","events_url":"https://api.github.com/users/ZhenLian/events{/privacy}","received_events_url":"https://api.github.com/users/ZhenLian/received_events","type":"User","site_admin":false},"labels":[{"id":170043423,"node_id":"MDU6TGFiZWwxNzAwNDM0MjM=","url":"https://api.github.com/repos/grpc/grpc/labels/area/security","name":"area/security","color":"efdb40","default":false,"description":null},{"id":166332908,"node_id":"MDU6TGFiZWwxNjYzMzI5MDg=","url":"https://api.github.com/repos/grpc/grpc/labels/lang/c++","name":"lang/c++","color":"fad8c7","default":false,"description":null},{"id":961832073,"node_id":"MDU6TGFiZWw5NjE4MzIwNzM=","url":"https://api.github.com/repos/grpc/grpc/labels/release%20notes:%20no","name":"release notes: no","color":"0f5f75","default":false,"description":"Indicates if PR should not be in release notes"}],"state":"open","locked":false,"assignee":{"login":"yihuazhang","id":26934891,"node_id":"MDQ6VXNlcjI2OTM0ODkx","avatar_url":"https://avatars2.githubusercontent.com/u/26934891?v=4","gravatar_id":"","url":"https://api.github.com/users/yihuazhang","html_url":"https://github.com/yihuazhang","followers_url":"https://api.github.com/users/yihuazhang/followers","following_url":"https://api.github.com/users/yihuazhang/following{/other_user}","gists_url":"https://api.github.com/users/yihuazhang/gists{/gist_id}","starred_url":"https://api.github.com/users/yihuazhang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yihuazhang/subscriptions","organizations_url":"https://api.github.com/users/yihuazhang/orgs","repos_url":"https://api.github.com/users/yihuazhang/repos","events_url":"https://api.github.com/users/yihuazhang/events{/privacy}","received_events_url":"https://api.github.com/users/yihuazhang/received_events","type":"User","site_admin":false},"assignees":[{"login":"yihuazhang","id":26934891,"node_id":"MDQ6VXNlcjI2OTM0ODkx","avatar_url":"https://avatars2.githubusercontent.com/u/26934891?v=4","gravatar_id":"","url":"https://api.github.com/users/yihuazhang","html_url":"https://github.com/yihuazhang","followers_url":"https://api.github.com/users/yihuazhang/followers","following_url":"https://api.github.com/users/yihuazhang/following{/other_user}","gists_url":"https://api.github.com/users/yihuazhang/gists{/gist_id}","starred_url":"https://api.github.com/users/yihuazhang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yihuazhang/subscriptions","organizations_url":"https://api.github.com/users/yihuazhang/orgs","repos_url":"https://api.github.com/users/yihuazhang/repos","events_url":"https://api.github.com/users/yihuazhang/events{/privacy}","received_events_url":"https://api.github.com/users/yihuazhang/received_events","type":"User","site_admin":false}],"milestone":null,"comments":4,"created_at":"2019-11-14T00:12:53Z","updated_at":"2019-11-25T22:10:05Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/grpc/grpc/pulls/21196","html_url":"https://github.com/grpc/grpc/pull/21196","diff_url":"https://github.com/grpc/grpc/pull/21196.diff","patch_url":"https://github.com/grpc/grpc/pull/21196.patch"},"body":"This is the second PR to expose ALTS context for users. It provides a utility function for users to get the AltsContext from AuthContext."},"comment":{"url":"https://api.github.com/repos/grpc/grpc/issues/comments/558362553","html_url":"https://github.com/grpc/grpc/pull/21196#issuecomment-558362553","issue_url":"https://api.github.com/repos/grpc/grpc/issues/21196","id":558362553,"node_id":"MDEyOklzc3VlQ29tbWVudDU1ODM2MjU1Mw==","user":{"login":"yang-g","id":10122250,"node_id":"MDQ6VXNlcjEwMTIyMjUw","avatar_url":"https://avatars0.githubusercontent.com/u/10122250?v=4","gravatar_id":"","url":"https://api.github.com/users/yang-g","html_url":"https://github.com/yang-g","followers_url":"https://api.github.com/users/yang-g/followers","following_url":"https://api.github.com/users/yang-g/following{/other_user}","gists_url":"https://api.github.com/users/yang-g/gists{/gist_id}","starred_url":"https://api.github.com/users/yang-g/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yang-g/subscriptions","organizations_url":"https://api.github.com/users/yang-g/orgs","repos_url":"https://api.github.com/users/yang-g/repos","events_url":"https://api.github.com/users/yang-g/events{/privacy}","received_events_url":"https://api.github.com/users/yang-g/received_events","type":"User","site_admin":false},"created_at":"2019-11-25T22:10:05Z","updated_at":"2019-11-25T22:10:05Z","author_association":"MEMBER","body":"`sysmacros.h` is a system header that does not have a namespace, and the macros are defined in global namespace. When `transport_security_common.proto` has `uint32 major;`, protobuf will generate a function `::google::protobuf::uint32 major();`, which will trigger the macro substitution and cause the build error.\r\n\r\nA simple way to verify this is to re-order the header and place `gtest.h` after `transport_security_common.pb.h` and the test can be built that way."}}
{"actor":{"display_login":"yang-g"}}