Skip to content

Recursive message support #74

Closed
@firdaus

Description

@firdaus

I am working on implementing a GRPC client which has a message such as this:

message Failure {
    ...
    Failure cause = 4;
    ...
}

Unfortunately it seems like betterproto attempts to build an object graph for this and this results in the following error:

RecursionError: maximum recursion depth exceeded while calling a Python object

I was wondering whether the project is planning on supporting this.

My plan in the short run is to just fork betterproto for my project and not set a default value for sub-messages in _get_field_default.

If the maintainers have a better approach to fixing this I would gladly offer my time in submitting a PR for it.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workinghigh prioritymediumMedium effort issue, can fit in a single PR

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions