Skip to content

Conversation

@anuraaga
Copy link
Collaborator

@anuraaga anuraaga commented Nov 14, 2025

As pyvoy was made with connect-python in mind, I guess it's not surprising it seems to handle the conformance tests fairly reliably.

Unfortunately, I haven't been able to get good results from granian and hypercorn so have gone ahead and removed them from the default runs, the wiring is still in place to try them again if things improve in the future.

@anuraaga anuraaga marked this pull request as draft November 14, 2025 06:47
Signed-off-by: Anuraag Agrawal <anuraaga@gmail.com>
@anuraaga anuraaga marked this pull request as ready for review November 14, 2025 14:57
Signed-off-by: Anuraag Agrawal <anuraaga@gmail.com>
# full duplex. There are no other popular WSGI servers that support HTTP/2, so in practice
# it cannot be supported. It is possible in theory following hyper-h2's example code in
# https://python-hyper.org/projects/hyper-h2/en/stable/wsgi-example.html though.
# TODO: Enable full-duplex in pyvoy
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This worked with some initial testing but I need to validate it some more

import sys

VERSION_CONFORMANCE = "v1.0.4"
VERSION_CONFORMANCE = "0bed9ca203aeda4e344edc442a4b2ede91726db5"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

want to create an issue over in https://github.com/connectrpc/conformance/issues to tag a new version for connectrpc/conformance#1047? cc @jhump @doriable

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anuraaga anuraaga merged commit 0efb7b0 into connectrpc:main Nov 15, 2025
23 checks passed
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

Successfully merging this pull request may close these issues.

2 participants