go-grpc-middleware icon indicating copy to clipboard operation
go-grpc-middleware copied to clipboard

Support for connect go

Open acbeni opened this issue 2 years ago • 5 comments

While working with another code generator ,connect go i missed the rich middleware env provided by this lib, and had the idea of forking this repo to modify the code to be suitable for the Interceptor interface in connect go.

But before that i wanted to ask if its healthier to refactor the common parts, and try to make this library compatible with both interfaces (the second being the grpc interceptor from grpc go) in which case i can contribute with PRs. or you would rather keep focus on grpc go and leave forks for other implementations.

Thanks in advance 🙏

acbeni avatar Aug 26 '23 20:08 acbeni

I think it'd be great to have some way to adapt the middleware in this repo for use with Connect. The library is already pretty dynamic so it might be possible to write a simple adapter.

CC @bwplotka @akshayjshah

johanbrandhorst avatar Aug 28 '23 16:08 johanbrandhorst

Makes sense. Would be nice to see how much work/code to maintain there is with some PoC MVP, but otherwise would be keen to have it working for both.

bwplotka avatar Aug 28 '23 17:08 bwplotka

Thank you for your encouraging replies, one question do you want me to do a POC on a fork and post link here, or how do you want me to proceed?

acbeni avatar Aug 29 '23 22:08 acbeni

Nah just open a PR and we can take it from there

johanbrandhorst avatar Aug 29 '23 22:08 johanbrandhorst

Sounds good to me 👍

acbeni avatar Aug 29 '23 22:08 acbeni