Cody Oss
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?
@shollyman please take a look.
@shollyman see my comment above
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...