micro--go-micro
3e885308a0
Fixes #2988. Brings 'golangci-lint run ./...' to zero issues (was ~373): - errcheck: explicitly ignore fire-and-forget calls with '_ =' (and a small errcheck.exclude-functions list for response writes — json Encoder.Encode, http ResponseWriter.Write, fmt.Fprint*); genuine cases handled. - unused: remove dead code (unexported decls and dead test helpers) and the imports they orphaned. - staticcheck: ST1005 error strings, ST1016 receiver names, S1000/S1017/S1019/ S1023 simplifications, SA4004/SA4006/SA4010 dead code, SA1021 net.IP.Equal, SA6002 (store *[]byte in sync.Pool). - govet: fix a context leak (lostcancel) in internal/util/mdns and move t.Fatal/Fatalf out of goroutines (testinggoroutine) in tests. - ineffassign, unconvert: mechanical fixes. CI: the Lint workflow now runs a blocking full-tree 'golangci-lint run' on pushes and PRs (dropped only-new-issues now that the tree is clean). Verified: go build, go vet, test compilation, and unit tests for the behaviourally-touched packages all pass. Claude-Session: https://claude.ai/code/session_01CmdEY7pYmV5zzwCjNJ4ykL Co-authored-by: Claude <noreply@anthropic.com>
105 行
1.6 KiB
Go
105 行
1.6 KiB
Go
package server
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"io"
|
|
"sync"
|
|
|
|
"go-micro.dev/v6/codec"
|
|
)
|
|
|
|
// Implements the Streamer interface.
|
|
type rpcStream struct {
|
|
err error
|
|
request Request
|
|
codec codec.Codec
|
|
context context.Context
|
|
id string
|
|
sync.RWMutex
|
|
closed bool
|
|
}
|
|
|
|
func (r *rpcStream) Context() context.Context {
|
|
return r.context
|
|
}
|
|
|
|
func (r *rpcStream) Request() Request {
|
|
return r.request
|
|
}
|
|
|
|
func (r *rpcStream) Send(msg interface{}) error {
|
|
r.Lock()
|
|
defer r.Unlock()
|
|
|
|
resp := codec.Message{
|
|
Target: r.request.Service(),
|
|
Method: r.request.Method(),
|
|
Endpoint: r.request.Endpoint(),
|
|
Id: r.id,
|
|
Type: codec.Response,
|
|
}
|
|
|
|
if err := r.codec.Write(&resp, msg); err != nil {
|
|
r.err = err
|
|
}
|
|
|
|
return nil
|
|
}
|
|
|
|
func (r *rpcStream) Recv(msg interface{}) error {
|
|
req := new(codec.Message)
|
|
req.Type = codec.Request
|
|
|
|
err := r.codec.ReadHeader(req, req.Type)
|
|
r.Lock()
|
|
defer r.Unlock()
|
|
if err != nil {
|
|
// discard body
|
|
_ = r.codec.ReadBody(nil)
|
|
r.err = err
|
|
return err
|
|
}
|
|
|
|
// check the error
|
|
if len(req.Error) > 0 {
|
|
// Check the client closed the stream
|
|
switch req.Error {
|
|
case errLastStreamResponse.Error():
|
|
// discard body
|
|
r.Unlock()
|
|
_ = r.codec.ReadBody(nil)
|
|
r.Lock()
|
|
r.err = io.EOF
|
|
return io.EOF
|
|
default:
|
|
return errors.New(req.Error)
|
|
}
|
|
}
|
|
|
|
// we need to stay up to date with sequence numbers
|
|
r.id = req.Id
|
|
r.Unlock()
|
|
err = r.codec.ReadBody(msg)
|
|
r.Lock()
|
|
if err != nil {
|
|
r.err = err
|
|
return err
|
|
}
|
|
|
|
return nil
|
|
}
|
|
|
|
func (r *rpcStream) Error() error {
|
|
r.RLock()
|
|
defer r.RUnlock()
|
|
return r.err
|
|
}
|
|
|
|
func (r *rpcStream) Close() error {
|
|
r.Lock()
|
|
defer r.Unlock()
|
|
r.closed = true
|
|
return r.codec.Close()
|
|
}
|