run: Details in error message dropped by googleapi.CheckResponse

[rileykarson]

I work on the Terraform Google Provider, where we often consume these generated service clients. We're using the googleapi.CheckResponse function to parse errors returned from GCP, however it's proved insufficient in this case.

The Cloud Run API returns a standard details object as defined by http://aip.dev/193. This gist provides an example error response I received. The details object isn't recognised, and only the shared fields (message / code) are properly parsed. Unfortunately, this API returns the string "The request has errors" as the message, resulting in the near-useless Error() string of googleapi: Error 400: The request has errors.

Given that this is now the standard error format per the AIP, I believe this library should parse the format into a useful error.

Original issue: hashicorp/terraform-provider-google#6099
Google-internal followup with Cloud Run: b/153896811

[codyoss]

Thanks for the report! I have ran into debugging woes in the past as well with the error messages generated. I will check out the aip and see what we can do.