yashisrani opened a new pull request, #3633: URL: https://github.com/apache/dubbo-go/pull/3633
## Description Replaces ad-hoc `fmt.Errorf` error returns with standard `*Error` values using `errorf(CodeInternal, ...)` in three functions: - `SetHeader` (line 252) - `SetTrailer` (line 275) - `SendHeader` (line 300) This ensures callers can use `errors.As` to inspect the error code and that errors are properly transmitted over the wire with the correct gRPC/Triple status code. ## Special notes for your reviewer: - No behavior change for existing callers — the error message is still descriptive - The error code `CodeInternal` is the appropriate classification for a missing handler context - All existing tests pass ## Checklist: - [x] I confirm the target branch is `develop` - [x] Code has passed local testing - [x] `go vet ./protocol/triple/triple_protocol/...` passes -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
