gofmt99%

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

• netcap/cmd/capture/main.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• netcap/maltego/transform_generation_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• netcap/cmd/transform/OpenFile.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• netcap/cmd/transform/defaults.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• netcap/cmd/transform/openConnectionInWireshark.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

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!

gocyclo91%

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.

• netcap/cmd/util/main.go
  • Line 32: warning: cyclomatic complexity 16 of function Run() is high (> 15) (gocyclo)

• netcap/io/utils.go
  • Line 111: warning: cyclomatic complexity 30 of function Dump() is high (> 15) (gocyclo)

• netcap/reassembly/tcpcheck.go
  • Line 173: warning: cyclomatic complexity 36 of function (*TCPSimpleFSM).CheckState() is high (> 15) (gocyclo)
  • Line 57: warning: cyclomatic complexity 18 of function (*TCPOptionCheck).Accept() is high (> 15) (gocyclo)

• netcap/decoder/stream/utils/saveFile.go
  • Line 50: warning: cyclomatic complexity 27 of function SaveFile() is high (> 15) (gocyclo)

• netcap/decoder/stream/utils/saveConversation.go
  • Line 45: warning: cyclomatic complexity 20 of function SaveConversation() is high (> 15) (gocyclo)

• netcap/decoder/stream/ssh/messages.go
  • Line 404: warning: cyclomatic complexity 35 of function Unmarshal() is high (> 15) (gocyclo)
  • Line 806: warning: cyclomatic complexity 30 of function decode() is high (> 15) (gocyclo)
  • Line 532: warning: cyclomatic complexity 23 of function marshalStruct() is high (> 15) (gocyclo)

• netcap/decoder/stream/software/software.go
  • Line 448: warning: cyclomatic complexity 40 of function WhatSoftwareHTTP() is high (> 15) (gocyclo)

• netcap/decoder/stream/abstract.go
  • Line 59: warning: cyclomatic complexity 17 of function InitAbstractDecoders() is high (> 15) (gocyclo)

• netcap/cmd/util/utils.go
  • Line 50: warning: cyclomatic complexity 17 of function checkFields() is high (> 15) (gocyclo)

• netcap/decoder/stream/http/httpMetaStore.go
  • Line 71: warning: cyclomatic complexity 18 of function updateHTTPStore() is high (> 15) (gocyclo)

• netcap/decoder/stream/pop3/pop3Reader.go
  • Line 249: warning: cyclomatic complexity 40 of function (*pop3Reader).processPOP3Conversation() is high (> 15) (gocyclo)

• netcap/decoder/stream/software/load.go
  • Line 28: warning: cyclomatic complexity 23 of function loadCmsDB() is high (> 15) (gocyclo)

• netcap/decoder/stream/stream.go
  • Line 52: warning: cyclomatic complexity 17 of function InitDecoders() is high (> 15) (gocyclo)

• netcap/decoder/stream/ssh/sshReader.go
  • Line 132: warning: cyclomatic complexity 24 of function (*sshReader).searchKexInit() is high (> 15) (gocyclo)

• netcap/dbs/clone.go
  • Line 31: warning: cyclomatic complexity 17 of function CloneDBs() is high (> 15) (gocyclo)

• netcap/decoder/stream/ssh/sshReader_test.go
  • Line 37: warning: cyclomatic complexity 16 of function TestParseSSHIdent() is high (> 15) (gocyclo)

• netcap/label/custom.go
  • Line 232: warning: cyclomatic complexity 30 of function customMap() is high (> 15) (gocyclo)

• netcap/label/tcp.go
  • Line 35: warning: cyclomatic complexity 23 of function labelTCP() is high (> 15) (gocyclo)

• netcap/label/ipv4.go
  • Line 35: warning: cyclomatic complexity 22 of function labelIPv4() is high (> 15) (gocyclo)

• netcap/label/ipv6.go
  • Line 35: warning: cyclomatic complexity 22 of function labelIPv6() is high (> 15) (gocyclo)

• netcap/label/suricata.go
  • Line 77: warning: cyclomatic complexity 27 of function Suricata() is high (> 15) (gocyclo)
  • Line 247: warning: cyclomatic complexity 24 of function parseSuricataFastLog() is high (> 15) (gocyclo)

• netcap/label/http.go
  • Line 22: warning: cyclomatic complexity 26 of function labelHTTP() is high (> 15) (gocyclo)

• netcap/label/layer.go
  • Line 37: warning: cyclomatic complexity 20 of function labelLayer() is high (> 15) (gocyclo)

• netcap/resolvers/service.go
  • Line 74: warning: cyclomatic complexity 25 of function InitServiceDB() is high (> 15) (gocyclo)

• netcap/cmd/capture/main.go
  • Line 43: warning: cyclomatic complexity 36 of function Run() is high (> 15) (gocyclo)

• netcap/cmd/agent/main.go
  • Line 40: warning: cyclomatic complexity 24 of function Run() is high (> 15) (gocyclo)

• netcap/decoder/stream/file/fileExtensions.go
  • Line 21: warning: cyclomatic complexity 75 of function ExtensionForContentType() is high (> 15) (gocyclo)

• netcap/decoder/stream/udp/udpStream.go
  • Line 146: warning: cyclomatic complexity 17 of function (*udpStream).decode() is high (> 15) (gocyclo)

• netcap/decoder/stream/service/serviceProbe.go
  • Line 365: warning: cyclomatic complexity 52 of function initServiceProbes() is high (> 15) (gocyclo)
  • Line 646: warning: cyclomatic complexity 32 of function clean() is high (> 15) (gocyclo)

• netcap/label/connection.go
  • Line 37: warning: cyclomatic complexity 31 of function labelConnections() is high (> 15) (gocyclo)

• netcap/label/tls.go
  • Line 35: warning: cyclomatic complexity 29 of function labelTLS() is high (> 15) (gocyclo)

• netcap/io/elastic.go
  • Line 376: warning: cyclomatic complexity 16 of function (*elasticWriter).sendBulk() is high (> 15) (gocyclo)

• netcap/cmd/proxy/netcapTransport.go
  • Line 43: warning: cyclomatic complexity 18 of function (*netcapTransport).RoundTrip() is high (> 15) (gocyclo)

• netcap/cmd/collect/main.go
  • Line 119: warning: cyclomatic complexity 24 of function udpServer() is high (> 15) (gocyclo)

• netcap/decoder/stream/credentials/imap.go
  • Line 25: warning: cyclomatic complexity 16 of function imapHarvester() is high (> 15) (gocyclo)

• netcap/dbs/index.go
  • Line 120: warning: cyclomatic complexity 62 of function IndexData() is high (> 15) (gocyclo)

• netcap/decoder/stream/utils/decode.go
  • Line 16: warning: cyclomatic complexity 20 of function DecodeConversation() is high (> 15) (gocyclo)

• netcap/decoder/stream/tcp/tcpConnection.go
  • Line 546: warning: cyclomatic complexity 21 of function CleanupReassembly() is high (> 15) (gocyclo)

• netcap/io/netcap.go
  • Line 30: warning: cyclomatic complexity 72 of function InitRecord() is high (> 15) (gocyclo)

• netcap/decoder/stream/mail/mailParsing.go
  • Line 215: warning: cyclomatic complexity 24 of function parseMailParts() is high (> 15) (gocyclo)

• netcap/label/udp.go
  • Line 35: warning: cyclomatic complexity 22 of function labelUDP() is high (> 15) (gocyclo)

• netcap/reassembly/assembler.go
  • Line 318: warning: cyclomatic complexity 32 of function (*Assembler).checkOverlap() is high (> 15) (gocyclo)
  • Line 181: warning: cyclomatic complexity 28 of function (*Assembler).AssembleWithContext() is high (> 15) (gocyclo)

golint97%

Golint is a linter for Go source code.

• netcap/decoder/stream/exploit/utils.go
  • Line 116: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)

• netcap/decoder/packet/gopacketDecoder.go
  • Line 59: warning: exported method GoPacketDecoder.PostInitFunc should have comment or be unexported (golint)
  • Line 63: warning: exported method GoPacketDecoder.DeInitFunc should have comment or be unexported (golint)
  • Line 67: warning: exported method GoPacketDecoder.GetName should have comment or be unexported (golint)
  • Line 71: warning: exported method GoPacketDecoder.SetWriter should have comment or be unexported (golint)
  • Line 75: warning: exported method GoPacketDecoder.GetType should have comment or be unexported (golint)
  • Line 79: warning: exported method GoPacketDecoder.GetDescription should have comment or be unexported (golint)
  • Line 83: warning: exported method GoPacketDecoder.NumRecords should have comment or be unexported (golint)
  • Line 268: warning: receiver name cd should be consistent with previous receiver name dec for GoPacketDecoder (golint)

• netcap/types/ethctp.go
  • Line 68: warning: receiver name a should be consistent with previous receiver name i for EthernetCTP (golint)

• netcap/types/sctp.go
  • Line 58: warning: receiver name u should be consistent with previous receiver name s for SCTP (golint)

• netcap/types/sip.go
  • Line 66: warning: receiver name u should be consistent with previous receiver name s for SIP (golint)

• netcap/types/snap.go
  • Line 50: warning: receiver name u should be consistent with previous receiver name s for SNAP (golint)

• netcap/cmd/split/main.go
  • Line 20: warning: exported function Run should have comment or be unexported (golint)

• netcap/dpi/dpi.go
  • Line 26: warning: should not use dot imports (golint)

• netcap/decoder/stream/ssh/common.go
  • Line 8: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)

• netcap/decoder/stream/ssh/messages.go
  • Line 38: warning: comment on exported type DisconnectMsg should be of the form "DisconnectMsg ..." (with optional leading article) (golint)
  • Line 53: warning: exported type KexInitMsg should have comment or be unexported (golint)
  • Line 74: warning: exported type KexDHInitMsg should have comment or be unexported (golint)
  • Line 80: warning: exported type KexECDHInitMsg should have comment or be unexported (golint)
  • Line 86: warning: exported type KexECDHReplyMsg should have comment or be unexported (golint)
  • Line 94: warning: exported type KexDHReplyMsg should have comment or be unexported (golint)
  • Line 103: warning: exported type KexDHGexGroupMsg should have comment or be unexported (golint)
  • Line 110: warning: exported type KexDHGexInitMsg should have comment or be unexported (golint)
  • Line 116: warning: exported type KexDHGexReplyMsg should have comment or be unexported (golint)
  • Line 124: warning: exported type KexDHGexRequestMsg should have comment or be unexported (golint)
  • Line 133: warning: exported type ServiceRequestMsg should have comment or be unexported (golint)
  • Line 140: warning: exported type ServiceAcceptMsg should have comment or be unexported (golint)
  • Line 147: warning: exported type UserAuthRequestMsg should have comment or be unexported (golint)
  • Line 154: warning: comment on exported type UserAuthSuccessMsg should be of the form "UserAuthSuccessMsg ..." (with optional leading article) (golint)
  • Line 161: warning: exported type UserAuthFailureMsg should have comment or be unexported (golint)
  • Line 172: warning: exported type UserAuthBannerMsg should have comment or be unexported (golint)
  • Line 184: warning: exported type UserAuthInfoRequestMsg should have comment or be unexported (golint)
  • Line 195: warning: exported type ChannelOpenMsg should have comment or be unexported (golint)
  • Line 208: warning: comment on exported type ChannelDataMsg should be of the form "ChannelDataMsg ..." (with optional leading article) (golint)
  • Line 218: warning: exported type ChannelOpenConfirmMsg should have comment or be unexported (golint)
  • Line 229: warning: exported type ChannelOpenFailureMsg should have comment or be unexported (golint)
  • Line 241: warning: exported const Prohibited should have comment (or a comment on this block) or be unexported (golint)
  • Line 264: warning: exported type ChannelRequestMsg should have comment or be unexported (golint)
  • Line 274: warning: exported type ChannelRequestSuccessMsg should have comment or be unexported (golint)
  • Line 281: warning: exported type ChannelRequestFailureMsg should have comment or be unexported (golint)
  • Line 288: warning: exported type ChannelCloseMsg should have comment or be unexported (golint)
  • Line 295: warning: exported type ChannelEOFMsg should have comment or be unexported (golint)
  • Line 302: warning: exported type GlobalRequestMsg should have comment or be unexported (golint)
  • Line 311: warning: exported type GlobalRequestSuccessMsg should have comment or be unexported (golint)
  • Line 318: warning: exported type GlobalRequestFailureMsg should have comment or be unexported (golint)
  • Line 325: warning: exported type WindowAdjustMsg should have comment or be unexported (golint)
  • Line 333: warning: exported type UserAuthPubKeyOkMsg should have comment or be unexported (golint)
  • Line 347: warning: exported type UserAuthGSSAPIToken should have comment or be unexported (golint)
  • Line 353: warning: exported type UserAuthGSSAPIMIC should have comment or be unexported (golint)
  • Line 360: warning: exported type UserAuthGSSAPIErrTok should have comment or be unexported (golint)
  • Line 367: warning: exported type UserAuthGSSAPIError should have comment or be unexported (golint)

• netcap/types/credentials.go
  • Line 66: warning: receiver name a should be consistent with previous receiver name c for Credentials (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign99%

IneffAssign detects ineffectual assignments in Go code.

• netcap/dbs/generate.go
  • Line 142: warning: ineffectual assignment to err (ineffassign)
  • Line 178: warning: ineffectual assignment to err (ineffassign)

• netcap/collector/log.go
  • Line 136: warning: ineffectual assignment to err (ineffassign)

• netcap/decoder/stream/service/serviceProbe.go
  • Line 605: warning: ineffectual assignment to arr (ineffassign)

• netcap/cmd/transform/LookupDHCPFingerprint.go
  • Line 97: warning: ineffectual assignment to path (ineffassign)

• netcap/cmd/util/mkpacket.go
  • Line 127: warning: ineffectual assignment to err (ineffassign)

misspell99%

Misspell Finds commonly misspelled English words

• netcap/decoder/stream/smtp/smtpReader_test.go
  • Line 48: warning: "addres" is a misspelling of "address" (misspell)
  • Line 67: warning: "addres" is a misspelling of "address" (misspell)

• netcap/decoder/stream/service/servicePorts.go
  • Line 641: warning: "routeros" is a misspelling of "routers" (misspell)
  • Line 1211: warning: "exportfs" is a misspelling of "exports" (misspell)

• netcap/decoder/stream/software/software.go
  • Line 269: warning: "likelyhood" is a misspelling of "likelihood" (misspell)