-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Messages with 'protobuf.Any' fields fail round trip dict construction #285
Labels
priority: p2
Moderately-important priority. Fix may not be included in next release.
type: bug
Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Comments
software-dov
added
type: bug
Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
priority: p2
Moderately-important priority. Fix may not be included in next release.
labels
Jan 25, 2022
software-dov
added a commit
to software-dov/gapic-generator-python
that referenced
this issue
Jan 25, 2022
* Use url_scheme when setting host name for REST transports * Expect correct exceptions in showcase tests * Fix the noxfile to fail if the precise interpreter required is missing * Temporarily bypass failing showcase tests for REST ** See googleapis/proto-plus-python#285 for details * In the showcase test setup, set the event loop once
software-dov
added a commit
to software-dov/gapic-generator-python
that referenced
this issue
Jan 25, 2022
* Use url_scheme when setting host name for REST transports * Expect correct exceptions in showcase tests * Fix the noxfile to fail if the precise interpreter required is missing * Temporarily bypass failing showcase tests for REST ** See googleapis/proto-plus-python#285 for details * In the showcase test setup, set the event loop once
software-dov
added a commit
to software-dov/gapic-generator-python
that referenced
this issue
Jan 27, 2022
* Use url_scheme when setting host name for REST transports * Expect correct exceptions in showcase tests * Fix the noxfile to fail if the precise interpreter required is missing * Temporarily bypass failing showcase tests for REST ** See googleapis/proto-plus-python#285 for details * In the showcase test setup, set the event loop once
yoshi-automation
added
🚨
This issue needs some love.
and removed
🚨
This issue needs some love.
labels
May 8, 2022
We are thinking about this issue, which affects multiple languages using protocol buffers over HTTP/JSON. We will continue providing updates, though the issue may not be completely resolved soon. |
Note that when this issue is resolved, some corresponding tests in |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
priority: p2
Moderately-important priority. Fix may not be included in next release.
type: bug
Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Consider the following snippet:
This fails because the dict representation for
any_pb2.Any
contains metadata thatproto.Message.__init__
andproto.Message.__setattr__
do not know how to handle/ignore properly.This blocks the use of the REST transport for any API method with a request that contains an
any_pb2.Any
fields due to transcoding logic.The text was updated successfully, but these errors were encountered: