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

Exporting memhttp / memhttptest #694

Open
oxisto opened this issue Feb 20, 2024 · 2 comments
Open

Exporting memhttp / memhttptest #694

oxisto opened this issue Feb 20, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@oxisto
Copy link

oxisto commented Feb 20, 2024

Is your feature request related to a problem? Please describe.

We are currently exploring migrating our project from the gRPC Go implementation to connect-go. Since our project uses several micro services, we have a lot of tests were we launch other microservices (in a defined state) to interact with our test target (or the other way around). For that we heavily rely on the bufconn package to avoid a lot of unnecessary (even though local) network traffic and network port management in the tests.

It seems that the internal packages memhttp / memhttptest offer a similar functionality. I wonder if there is an intent to publish these packages as an official testing solution?

Describe the solution you'd like

Export the memhttp package.

@jhump
Copy link
Member

jhump commented Feb 20, 2024

@oxisto, the internal package here was largely based on https://github.com/akshayjshah/memhttp. We are in fact considering something like this as exported API available in the connectrpc GitHub organization, but until then you can use the above 3rd-party module.

@oxisto
Copy link
Author

oxisto commented Feb 20, 2024

@oxisto, the internal package here was largely based on https://github.com/akshayjshah/memhttp. We are in fact considering something like this as exported API available in the connectrpc GitHub organization, but until then you can use the above 3rd-party module.

Ah excellent, thanks! Having this within the connectrpc org would be awesome, but this will do for the migration period.

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

No branches or pull requests

2 participants