search

connectrpc / connect-go

The Go implementation of Connect: Protobuf RPC that works.
https://connectrpc.com
Apache License 2.0
2.76k stars 91 forks source link

Fix ErrorWriter IsSupportedCheck with required connect protocol option #700

Closed emcfarlane closed 4 months ago

emcfarlane commented 4 months ago

This PR fixes ErrorWriter to correctly return unsupported protocol if the option WithRequireConnectProtocolHeader is set and the header or query value isn't include in the request. It will now correctly return unsupported to ensure fallback options can process the error.

Fixes #699