Cody Oss

Results 277 comments of Cody Oss

Correction from the previous comment. This is currently only affecting targets that use LROs AND import any well known proto type. This kinda makes sense as it is the update...

Said logic: https://github.com/googleapis/google-cloud-go/blob/d8a53d60472e8066507559478ce5554cfd794339/auth/httptransport/transport.go#L71-L75 and https://github.com/googleapis/google-cloud-go/blob/d8a53d60472e8066507559478ce5554cfd794339/auth/grpctransport/grpctransport.go#L268-L272

I think the extra building we do in CI should now catch what this would. Do we think this is still needed?

Hey @12ya I see that you have made a couple PRs. In general we ask contributors to make an issue first to discuss potential changes. There can we a lot...