StackDriver Error reporting payload key is wrong

[bmorgan21]

https://github.com/GoogleCloudPlatform/google-cloud-python/blob/0.24.0/error_reporting/google/cloud/error_reporting/client.py#L227

I believe this like should set the payload context key to httpRequest and NOT httpContext. This would make it match the spec.

https://cloud.google.com/error-reporting/docs/formatting-error-messages

Right now the report_exception call is throwing an error when using grpc and not populating the data with the http api usage.

google.protobuf.json_format.ParseError: Message type "google.devtools.clouderrorreporting.v1beta1.ErrorContext" has no field named "httpContext".

[bmorgan21]

here is a code snippet that at least gets the data to show up.

class MyClient(error_reporting.Client):

        def _build_error_report(self, *args, **kwargs):
            payload = super(MyClient, self)._build_error_report(*args, **kwargs)

            if 'context' in payload:
                data = payload['context'].pop('httpContext', None)
                if data:
                    payload['context']['httpRequest'] = data

            return payload

[dhermes]

Thanks for reporting @bmorgan21, this was fixed in #3299. It seems we should get a release out soon?

[bmorgan21]

thanks.

is there a dev build I can use? vs having to patch the broken code?

[lukesneeringer]

@dhermes Go ahead and cut a release. Thanks!

[dhermes]

@bmorgan21 Kicked off a build

https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/1652

that will push the new release to

https://pypi.python.org/pypi/google-cloud-error-reporting/0.24.1

[bmorgan21]

@dhermes thank you for the quick turnaround.