Preparing report...

Report for github.com/bfenetworks/bfe

A    Great!    Found 291 issues across 488 files

Tweet

gofmt97%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command


go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!


gocyclo92%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

    • bfe/bfe_bufio/bufio_test.go
    • Line 520: warning: cyclomatic complexity 29 of function TestPeek() is high (> 15) (gocyclo)
    • Line 274: warning: cyclomatic complexity 16 of function TestUnreadRuneError() is high (> 15) (gocyclo)
    • bfe/bfe_tls/handshake_messages.go
    • Line 338: warning: cyclomatic complexity 51 of function (*clientHelloMsg).unmarshal() is high (> 15) (gocyclo)
    • Line 110: warning: cyclomatic complexity 30 of function (*clientHelloMsg).marshal() is high (> 15) (gocyclo)
    • Line 676: warning: cyclomatic complexity 26 of function (*serverHelloMsg).unmarshal() is high (> 15) (gocyclo)
    • Line 560: warning: cyclomatic complexity 18 of function (*serverHelloMsg).marshal() is high (> 15) (gocyclo)
    • Line 47: warning: cyclomatic complexity 17 of function (*clientHelloMsg).equal() is high (> 15) (gocyclo)
    • Line 1215: warning: cyclomatic complexity 16 of function (*certificateRequestMsg).unmarshal() is high (> 15) (gocyclo)
    • bfe/bfe_http/transfer.go
    • Line 276: warning: cyclomatic complexity 25 of function readTransfer() is high (> 15) (gocyclo)
    • Line 52: warning: cyclomatic complexity 19 of function newTransferWriter() is high (> 15) (gocyclo)
    • bfe/bfe_tls/handshake_client.go
    • Line 226: warning: cyclomatic complexity 51 of function (*clientHandshakeState).doFullHandshake() is high (> 15) (gocyclo)
    • Line 46: warning: cyclomatic complexity 40 of function (*Conn).clientHandshake() is high (> 15) (gocyclo)
    • bfe/bfe_http/cookie.go
    • Line 135: warning: cyclomatic complexity 28 of function readSetCookies() is high (> 15) (gocyclo)
    • Line 368: warning: cyclomatic complexity 21 of function isCookieDomainName() is high (> 15) (gocyclo)
    • bfe/bfe_http2/server.go
    • Line 2289: warning: cyclomatic complexity 30 of function (*responseWriterState).writeChunk() is high (> 15) (gocyclo)
    • Line 1886: warning: cyclomatic complexity 26 of function (*serverConn).newWriterAndRequest() is high (> 15) (gocyclo)
    • Line 1568: warning: cyclomatic complexity 18 of function (*serverConn).processData() is high (> 15) (gocyclo)
    • Line 831: warning: cyclomatic complexity 17 of function (*serverConn).serve() is high (> 15) (gocyclo)
    • bfe/bfe_server/reverseproxy.go
    • Line 568: warning: cyclomatic complexity 34 of function (*ReverseProxy).ServeHTTP() is high (> 15) (gocyclo)
    • Line 246: warning: cyclomatic complexity 22 of function (*ReverseProxy).clusterInvoke() is high (> 15) (gocyclo)
    • bfe/bfe_tls/handshake_server.go
    • Line 153: warning: cyclomatic complexity 49 of function (*serverHandshakeState).readClientHello() is high (> 15) (gocyclo)
    • Line 580: warning: cyclomatic complexity 35 of function (*serverHandshakeState).doFullHandshake() is high (> 15) (gocyclo)
    • Line 465: warning: cyclomatic complexity 24 of function (*serverHandshakeState).checkForResumption() is high (> 15) (gocyclo)
    • Line 951: warning: cyclomatic complexity 17 of function (*Conn).tryCipherSuite() is high (> 15) (gocyclo)
    • Line 56: warning: cyclomatic complexity 17 of function (*Conn).serverHandshake() is high (> 15) (gocyclo)
    • bfe/bfe_tls/conn.go
    • Line 733: warning: cyclomatic complexity 44 of function (*Conn).readRecord() is high (> 15) (gocyclo)
    • Line 1045: warning: cyclomatic complexity 22 of function (*Conn).readHandshake() is high (> 15) (gocyclo)
    • Line 316: warning: cyclomatic complexity 19 of function (*halfConn).decrypt() is high (> 15) (gocyclo)
    • Line 1157: warning: cyclomatic complexity 18 of function (*Conn).Read() is high (> 15) (gocyclo)
    • Line 640: warning: cyclomatic complexity 17 of function convertSSLv2ClientHello() is high (> 15) (gocyclo)
    • Line 950: warning: cyclomatic complexity 16 of function (*Conn).writeRecord() is high (> 15) (gocyclo)
    • bfe/bfe_http/transport.go
    • Line 781: warning: cyclomatic complexity 31 of function (*persistConn).readLoop() is high (> 15) (gocyclo)
    • Line 532: warning: cyclomatic complexity 19 of function (*Transport).dialConn() is high (> 15) (gocyclo)
    • Line 635: warning: cyclomatic complexity 18 of function useProxy() is high (> 15) (gocyclo)
    • bfe/bfe_tls/key_agreement.go
    • Line 327: warning: cyclomatic complexity 22 of function (*ecdheKeyAgreement).processServerKeyExchange() is high (> 15) (gocyclo)
    • Line 216: warning: cyclomatic complexity 19 of function (*ecdheKeyAgreement).generateServerKeyExchange() is high (> 15) (gocyclo)

golint0%

Golint is a linter for Go source code.

An error occurred while running this test (exit status 3)


license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign98%

IneffAssign detects ineffectual assignments in Go code.


misspell0%

Misspell Finds commonly misspelled English words

An error occurred while running this test (exit status 2)