gapic-generator-python

gapic-generator-python copied to clipboard

fix rendering in internal READMEs

1

Some of the inner READMEs, such as [this one](https://github.com/googleapis/google-cloud-python/blob/adea041c1d43b94d11ff9d4d5ba13c0d7898cac2/packages/google-cloud-speech/google/cloud/speech_v1/services/speech/transports/README.rst?plain=1#L2), are not rendered as intended. We need to fix the RST.

vchudnov-g

Update the logging section of README to refer to a generated library name instead of referring to a generic `library` name.

ohmayr

[logging] REST payloads are over-stringified and lose JSON structure

1

In the current version of #2270, the `httpResponse.payload` field is over-stringified: it's a single string with string-encoded JSON, rather than JSON object with the various fields. Since the response is...

vchudnov-g

Implement a test against Showcase returning metadata as bytes

2

Update showcase to test metadata with bytes and add a unit test to confirm this behaviour.

ohmayr

Extra fields are only included in the structured log format. We can update the code to append these extra fields directly in the message so that they appear if a...

ohmayr

We should be able to add a Github action to flag when there are spelling mistakes in pull requests. It's not clear which tool we should use for spell checking...

parthea

auto-gen snippets exceeds Python recursion limit in some cases

1

Generating the GAPIC for compute alpha (using the [disco-to-proto3 converter](https://github.com/googleapis/disco-to-proto3-converter) to create proto3s from the [Discovery file](https://www.googleapis.com/discovery/v1/apis/compute/alpha/rest)) results in a `RecursionError: maximum recursion depth exceeded while calling a Python object`....

vchudnov-g

We check the type of request / response i.e. `proto.Message` or `google.protobuf.message.Message` to appropriately convert the message to json. investigate if there's a better way or if we should simply...

ohmayr

We're logging rest headers as part of the http response and separately as a metadata field. This seems repetitive and needs to be cleaned up.

ohmayr

Investigate and add a debug log event before intercepting a `REST`. Look for this issue number to find where to add this log event.

ohmayr