LLKennedy/httpgrpc

View on GitHub
proxy/streamdual.go

Summary

Maintainability
A
1 hr
Test Coverage

Method Server.handleDualStream has 59 lines of code (exceeds 50 allowed). Consider refactoring.
Open

func (s *Server) handleDualStream(ctx context.Context, procType reflect.Type, caller reflect.Value, srv httpapi.ExposedService_ProxyStreamServer) (err error) {
    defer func() {
        r := recover()
        if r != nil {
            err = status.Errorf(codes.Internal, "caught panic for dual stream: %v", r)
Severity: Minor
Found in proxy/streamdual.go - About 1 hr to fix

    Method Server.handleDualStream has 5 return statements (exceeds 4 allowed).
    Open

    func (s *Server) handleDualStream(ctx context.Context, procType reflect.Type, caller reflect.Value, srv httpapi.ExposedService_ProxyStreamServer) (err error) {
        defer func() {
            r := recover()
            if r != nil {
                err = status.Errorf(codes.Internal, "caught panic for dual stream: %v", r)
    Severity: Major
    Found in proxy/streamdual.go - About 35 mins to fix

      There are no issues that match your filters.

      Category
      Status