Skip to content
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

Docs say status_details_codec defaults to ProtoStatusDetailsCodec() but actual default is None #198

Open
stan-stately opened this issue Aug 30, 2024 · 0 comments

Comments

@stan-stately
Copy link

This means that by default the status details aren't included which is kind of confusing.

ref:

grpclib/grpclib/client.py

Lines 624 to 647 in 62f968a

status_details_codec: Optional[StatusDetailsCodecBase] = None,
ssl: Union[
None, bool, "_ssl.SSLContext", "_ssl.DefaultVerifyPaths"
] = None,
config: Optional[Configuration] = None,
):
"""Initialize connection to the server
:param host: server host name.
:param port: server port number.
:param loop: (deprecated) asyncio-compatible event loop
:param path: server socket path. If specified, host and port should be
omitted (must be None).
:param codec: instance of a codec to encode and decode messages,
if omitted ``ProtoCodec`` is used by default
:param status_details_codec: instance of a status details codec to
decode error details in a trailing metadata, if omitted
``ProtoStatusDetailsCodec`` is used by default

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant