gofmt99%

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

• peer-calls/server/udpmux/udpmux_test.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!

gocyclo96%

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.

• peer-calls/server/udpmux/udpmux.go
  • Line 104: warning: cyclomatic complexity 22 of function (*UDPMux).startLoop() is high (> 15) (gocyclo)

• peer-calls/server/servertransport/metadata_transport.go
  • Line 127: warning: cyclomatic complexity 16 of function (*MetadataTransport).startReadLoop() is high (> 15) (gocyclo)

• peer-calls/server/cli/play.go
  • Line 229: warning: cyclomatic complexity 33 of function (*playHandler).handleMessages() is high (> 15) (gocyclo)

• peer-calls/server/udptransport2/factory.go
  • Line 227: warning: cyclomatic complexity 51 of function (*Factory).start() is high (> 15) (gocyclo)

• peer-calls/server/udptransport2/manager.go
  • Line 68: warning: cyclomatic complexity 36 of function (*Manager).start() is high (> 15) (gocyclo)

golint42%

Golint is a linter for Go source code.

• peer-calls/server/transport/track_event.go
  • Line 3: warning: exported type TrackEventType should have comment or be unexported (golint)
  • Line 6: warning: exported const TrackEventTypeAdd should have comment (or a comment on this block) or be unexported (golint)

• peer-calls/server/sfu/stats/scheduler.go
  • Line 377: warning: don't use underscores in Go names; struct field we_sent should be weSent (golint)
  • Line 382: warning: don't use underscores in Go names; struct field avg_rtcp_size should be avgRtcpSize (golint)
  • Line 393: warning: don't use underscores in Go names; struct field rtcp_bw should be rtcpBw (golint)
  • Line 448: warning: don't use underscores in Go names; var rtcp_min_time should be rtcpMinTime (golint)
  • Line 451: warning: don't use underscores in Go names; var rtcp_bw should be rtcpBw (golint)
  • Line 460: warning: don't use underscores in Go names; var rtcp_min_time should be rtcpMinTime (golint)

• peer-calls/server/udptransport2/factory.go
  • Line 26: warning: exported type Factory should have comment or be unexported (golint)
  • Line 40: warning: exported type FactoryParams should have comment or be unexported (golint)
  • Line 61: warning: exported function NewFactory should have comment or be unexported (golint)
  • Line 522: warning: exported method Factory.TransportsChannel should have comment or be unexported (golint)
  • Line 526: warning: exported method Factory.CreateTransport should have comment or be unexported (golint)
  • Line 544: warning: exported method Factory.CloseTransport should have comment or be unexported (golint)
  • Line 562: warning: exported method Factory.Done should have comment or be unexported (golint)
  • Line 566: warning: exported method Factory.Close should have comment or be unexported (golint)

• peer-calls/server/cli/cli.go
  • Line 11: warning: exported type Props should have comment or be unexported (golint)
  • Line 18: warning: exported function Exec should have comment or be unexported (golint)

• peer-calls/server/wrtcsignaller.go
  • Line 13: warning: exported type Signaller should have comment or be unexported (golint)
  • Line 30: warning: exported function NewSignaller should have comment or be unexported (golint)
  • Line 62: warning: comment on exported method Signaller.Done should be of the form "Done ..." (golint)
  • Line 68: warning: exported method Signaller.SignalChannel should have comment or be unexported (golint)
  • Line 103: warning: exported method Signaller.Initiator should have comment or be unexported (golint)
  • Line 141: warning: exported method Signaller.Close should have comment or be unexported (golint)
  • Line 184: warning: exported method Signaller.Signal should have comment or be unexported (golint)
  • Line 330: warning: comment on exported method Signaller.SendTransceiverRequest should be of the form "SendTransceiverRequest ..." (golint)
  • Line 348: warning: comment on exported method Signaller.Negotiate should be of the form "Negotiate ..." (golint)

• peer-calls/server/wssub.go
  • Line 17: warning: exported type WSS should have comment or be unexported (golint)
  • Line 22: warning: exported function NewWSS should have comment or be unexported (golint)
  • Line 29: warning: exported type WebsocketContext should have comment or be unexported (golint)

• peer-calls/server/multierr/sync.go
  • Line 20: warning: comment on exported method Sync.Err should be of the form "Err ..." (golint)

• peer-calls/server/codecs/registry.go
  • Line 12: warning: exported var ErrUnsupportedMimeType should have comment or be unexported (golint)
  • Line 14: warning: exported type Registry should have comment or be unexported (golint)
  • Line 19: warning: exported type Props should have comment or be unexported (golint)
  • Line 24: warning: exported type HeaderExtension should have comment or be unexported (golint)
  • Line 31: warning: exported const PayloadTypeOpus should have comment (or a comment on this block) or be unexported (golint)
  • Line 45: warning: exported function NewRegistryDefault should have comment or be unexported (golint)
  • Line 163: warning: exported type MatchType should have comment or be unexported (golint)
  • Line 166: warning: exported const MatchNone should have comment (or a comment on this block) or be unexported (golint)
  • Line 171: warning: comment on exported method Registry.FuzzySearch should be of the form "FuzzySearch ..." (golint)
  • Line 199: warning: exported method Registry.RTPHeaderExtensionsForMimeType should have comment or be unexported (golint)
  • Line 215: warning: exported method Registry.InterceptorParamsForCodec should have comment or be unexported (golint)
  • Line 256: warning: exported function TypeFromMimeType should have comment or be unexported (golint)
  • Line 264: warning: exported type InterceptorParams should have comment or be unexported (golint)

• peer-calls/server/pubsub/pubsub.go
  • Line 266: warning: exported type TrackProps should have comment or be unexported (golint)
  • Line 272: warning: comment on exported method PubSub.TrackPropsByTrackID should be of the form "TrackPropsByTrackID ..." (golint)

• peer-calls/server/udpmux/errors.go
  • Line 6: warning: exported var ErrConnAlreadyExists should have comment or be unexported (golint)

• peer-calls/server/message/message.go
  • Line 8: warning: exported type Message should have comment or be unexported (golint)
  • Line 18: warning: exported function NewReady should have comment or be unexported (golint)
  • Line 28: warning: exported function NewHangUp should have comment or be unexported (golint)
  • Line 38: warning: exported function NewRoomJoin should have comment or be unexported (golint)
  • Line 48: warning: exported function NewRoomLeave should have comment or be unexported (golint)
  • Line 58: warning: exported function NewUsers should have comment or be unexported (golint)
  • Line 68: warning: exported function NewPubTrack should have comment or be unexported (golint)
  • Line 78: warning: exported function NewSubTrack should have comment or be unexported (golint)
  • Line 88: warning: exported function NewSignal should have comment or be unexported (golint)
  • Line 98: warning: exported type UserSignal should have comment or be unexported (golint)
  • Line 125: warning: exported type RoomJoin should have comment or be unexported (golint)
  • Line 130: warning: exported type Type should have comment or be unexported (golint)
  • Line 133: warning: exported const TypeHangUp should have comment (or a comment on this block) or be unexported (golint)
  • Line 147: warning: exported type HangUp should have comment or be unexported (golint)
  • Line 151: warning: exported type Ready should have comment or be unexported (golint)
  • Line 155: warning: exported type Ping should have comment or be unexported (golint)
  • Line 157: warning: comment on exported type Users should be of the form "Users ..." (with optional leading article) (golint)
  • Line 192: warning: exported type SubTrack should have comment or be unexported (golint)

• peer-calls/server/message/json.go
  • Line 10: warning: exported var ErrUnknownMessageType should have comment or be unexported (golint)
  • Line 12: warning: exported type JSON should have comment or be unexported (golint)
  • Line 20: warning: exported method Message.MarshalJSON should have comment or be unexported (golint)
  • Line 73: warning: exported method Message.UnmarshalJSON should have comment or be unexported (golint)

• peer-calls/server/basen/basen.go
  • Line 10: warning: exported const AlphabetBase16 should have comment (or a comment on this block) or be unexported (golint)

• peer-calls/server/transport/simple_track.go
  • Line 10: warning: exported type SimpleTrack should have comment or be unexported (golint)
  • Line 18: warning: exported function NewSimpleTrack should have comment or be unexported (golint)
  • Line 29: warning: exported method SimpleTrack.SimpleTrack should have comment or be unexported (golint)
  • Line 33: warning: exported method SimpleTrack.PeerID should have comment or be unexported (golint)
  • Line 37: warning: exported method SimpleTrack.TrackID should have comment or be unexported (golint)
  • Line 41: warning: exported method SimpleTrack.Codec should have comment or be unexported (golint)
  • Line 45: warning: exported method SimpleTrack.MarshalJSON should have comment or be unexported (golint)
  • Line 53: warning: exported method SimpleTrack.UnmarshalJSON should have comment or be unexported (golint)

• peer-calls/server/sfu/sub_params.go
  • Line 7: warning: exported type SubParams should have comment or be unexported (golint)

• peer-calls/server/servertransport/server_track.go
  • Line 10: warning: exported type ServerTrack should have comment or be unexported (golint)
  • Line 19: warning: exported method ServerTrack.Sub should have comment or be unexported (golint)
  • Line 27: warning: exported method ServerTrack.Unsub should have comment or be unexported (golint)

• peer-calls/server/cli/root.go
  • Line 7: warning: exported function NewRootCommand should have comment or be unexported (golint)

• peer-calls/server/atomic/bool.go
  • Line 5: warning: exported type Bool should have comment or be unexported (golint)
  • Line 9: warning: exported method Bool.CompareAndSwap should have comment or be unexported (golint)
  • Line 26: warning: exported method Bool.Set should have comment or be unexported (golint)
  • Line 32: warning: exported method Bool.Get should have comment or be unexported (golint)

• peer-calls/server/stringmux/stringmux.go
  • Line 12: warning: exported const DefaultMTU should have comment or be unexported (golint)
  • Line 14: warning: exported type StringMux should have comment or be unexported (golint)
  • Line 26: warning: exported type Params should have comment or be unexported (golint)
  • Line 34: warning: exported function New should have comment or be unexported (golint)
  • Line 264: warning: exported method StringMux.GetConn should have comment or be unexported (golint)
  • Line 285: warning: exported method StringMux.Close should have comment or be unexported (golint)
  • Line 300: warning: exported method StringMux.LocalAddr should have comment or be unexported (golint)
  • Line 304: warning: exported method StringMux.RemoteAddr should have comment or be unexported (golint)
  • Line 308: warning: exported method StringMux.Done should have comment or be unexported (golint)

• peer-calls/server/test/logger.go
  • Line 8: warning: exported function NewLogger should have comment or be unexported (golint)

• peer-calls/server/cli/play/rtcp_writer.go
  • Line 11: warning: exported type RTCPWriter should have comment or be unexported (golint)
  • Line 16: warning: exported type RTCPWriterParams should have comment or be unexported (golint)
  • Line 22: warning: exported function NewRTCPWriter should have comment or be unexported (golint)
  • Line 32: warning: exported method RTCPWriter.WriteRTCP should have comment or be unexported (golint)
  • Line 49: warning: exported method RTCPWriter.Close should have comment or be unexported (golint)

• peer-calls/server/sfu/jitter.go
  • Line 9: warning: exported type JitterHandler should have comment or be unexported (golint)
  • Line 15: warning: exported type NackHandler should have comment or be unexported (golint)
  • Line 21: warning: exported function NewJitterHandler should have comment or be unexported (golint)
  • Line 29: warning: exported function NewJitterNackHandler should have comment or be unexported (golint)
  • Line 42: warning: comment on exported method NackHandler.HandleNack should be of the form "HandleNack ..." (golint)
  • Line 99: warning: exported method NackHandler.HandleRTP should have comment or be unexported (golint)
  • Line 103: warning: exported method NackHandler.RemoveBuffer should have comment or be unexported (golint)
  • Line 107: warning: exported type NoopNackHandler should have comment or be unexported (golint)
  • Line 109: warning: exported method NoopNackHandler.HandleNack should have comment or be unexported (golint)
  • Line 114: warning: exported method NoopNackHandler.HandleRTP should have comment or be unexported (golint)
  • Line 118: warning: exported method NoopNackHandler.RemoveBuffer should have comment or be unexported (golint)

• peer-calls/server/sfu/sfu.go
  • Line 1: warning: package comment should be of the form "Package sfu ..." (golint)

• peer-calls/server/udptransport2/transport.go
  • Line 13: warning: exported type Transport should have comment or be unexported (golint)
  • Line 20: warning: exported function NewTransport should have comment or be unexported (golint)
  • Line 51: warning: exported method Transport.CloseWrite should have comment or be unexported (golint)
  • Line 55: warning: comment on exported method Transport.StreamID should be of the form "StreamID ..." (golint)

• peer-calls/server/sfu.go
  • Line 17: warning: exported const IOSH264Fmtp should have comment or be unexported (golint)
  • Line 23: warning: exported function NewSFUHandler should have comment or be unexported (golint)
  • Line 37: warning: exported type SFU should have comment or be unexported (golint)
  • Line 84: warning: exported type SocketHandler should have comment or be unexported (golint)
  • Line 96: warning: exported function NewSocketHandler should have comment or be unexported (golint)
  • Line 114: warning: exported method SocketHandler.HandleMessage should have comment or be unexported (golint)
  • Line 137: warning: exported method SocketHandler.HangUp should have comment or be unexported (golint)

• peer-calls/server/transport/track_info.go
  • Line 3: warning: exported type TrackWithMID should have comment or be unexported (golint)
  • Line 9: warning: exported function NewTrackWithMID should have comment or be unexported (golint)
  • Line 13: warning: exported method TrackWithMID.MID should have comment or be unexported (golint)

• peer-calls/server/sfu/stats/ntp.go
  • Line 16: warning: exported type NTPTime should have comment or be unexported (golint)
  • Line 18: warning: exported function NewNTPTime should have comment or be unexported (golint)
  • Line 35: warning: exported method NTPTime.Time should have comment or be unexported (golint)
  • Line 50: warning: exported method NTPTime.Middle should have comment or be unexported (golint)

• peer-calls/server/pubsub/remb.go
  • Line 108: warning: exported method BitrateEstimator.RemoveClientBitrate should have comment or be unexported (golint)

• peer-calls/server/servertransport/media_stream.go
  • Line 19: warning: exported type MediaStream should have comment or be unexported (golint)
  • Line 54: warning: exported type BufferFactory should have comment or be unexported (golint)
  • Line 60: warning: exported type MediaStreamParams should have comment or be unexported (golint)
  • Line 70: warning: exported function NewMediaStream should have comment or be unexported (golint)
  • Line 143: warning: exported method MediaStream.GetOrCreateBuffer should have comment or be unexported (golint)
  • Line 156: warning: exported method MediaStream.RemoveBuffer should have comment or be unexported (golint)
  • Line 169: warning: exported method MediaStream.Writer should have comment or be unexported (golint)
  • Line 266: warning: exported method MediaStream.WriteRTCP should have comment or be unexported (golint)
  • Line 287: warning: exported method MediaStream.Close should have comment or be unexported (golint)

• peer-calls/server/iceauth.go
  • Line 11: warning: exported function GetICEAuthServers should have comment or be unexported (golint)

• peer-calls/server/mesh.go
  • Line 14: warning: exported type ReadyMessage should have comment or be unexported (golint)
  • Line 19: warning: exported function NewMeshHandler should have comment or be unexported (golint)

• peer-calls/server/uuid/uuid.go
  • Line 10: warning: exported function New should have comment or be unexported (golint)

• peer-calls/server/sfu/stats/source.go
  • Line 139: warning: comment on exported method Source.ReceptionReport should be of the form "ReceptionReport ..." (golint)
  • Line 219: warning: exported method Source.HandleSenderReport should have comment or be unexported (golint)
  • Line 223: warning: exported method Source.HandleRTP should have comment or be unexported (golint)

• peer-calls/server/logger/logger.go
  • Line 38: warning: exported type Factory should have comment or be unexported (golint)
  • Line 84: warning: exported function NewFromEnv should have comment or be unexported (golint)

• peer-calls/server/pionlogger/pionlogger.go
  • Line 14: warning: exported type Factory should have comment or be unexported (golint)
  • Line 18: warning: exported function NewFactory should have comment or be unexported (golint)
  • Line 23: warning: exported method Factory.NewLogger should have comment or be unexported (golint)

• peer-calls/server/logger/message.go
  • Line 5: warning: exported type Message should have comment or be unexported (golint)

• peer-calls/server/udptransport2/manager.go
  • Line 16: warning: exported type Manager should have comment or be unexported (golint)
  • Line 28: warning: exported type ManagerParams should have comment or be unexported (golint)
  • Line 37: warning: exported function NewManager should have comment or be unexported (golint)
  • Line 307: warning: exported method Manager.GetFactory should have comment or be unexported (golint)
  • Line 329: warning: exported method Manager.Factories should have comment or be unexported (golint)
  • Line 344: warning: exported method Manager.Close should have comment or be unexported (golint)
  • Line 357: warning: exported type NewFactoryResponse should have comment or be unexported (golint)
  • Line 363: warning: exported method NewFactoryResponse.Result should have comment or be unexported (golint)

• peer-calls/server/servertransport/transport.go
  • Line 19: warning: comment on exported const ReceiveMTU should be of the form "ReceiveMTU ..." (golint)
  • Line 24: warning: exported var ErrNoData should have comment or be unexported (golint)
  • Line 62: warning: exported type Params should have comment or be unexported (golint)
  • Line 71: warning: exported function New should have comment or be unexported (golint)
  • Line 121: warning: exported method Transport.ClientID should have comment or be unexported (golint)
  • Line 125: warning: exported method Transport.Done should have comment or be unexported (golint)
  • Line 129: warning: exported method Transport.Close should have comment or be unexported (golint)
  • Line 145: warning: exported method Transport.Type should have comment or be unexported (golint)

• peer-calls/server/mux.go
  • Line 39: warning: exported type Mux should have comment or be unexported (golint)
  • Line 51: warning: exported type TracksManager should have comment or be unexported (golint)
  • Line 64: warning: exported type RoomManager should have comment or be unexported (golint)
  • Line 69: warning: exported function NewMux should have comment or be unexported (golint)

• peer-calls/server/render.go
  • Line 13: warning: exported type Renderer should have comment or be unexported (golint)
  • Line 22: warning: exported function NewRenderer should have comment or be unexported (golint)
  • Line 32: warning: exported type PageHandler should have comment or be unexported (golint)
  • Line 37: warning: exported method Renderer.Render should have comment or be unexported (golint)

• peer-calls/server/template.go
  • Line 9: warning: exported type Templates should have comment or be unexported (golint)
  • Line 11: warning: exported method Templates.Get should have comment or be unexported (golint)
  • Line 17: warning: exported function ParseTemplates should have comment or be unexported (golint)

• peer-calls/server/command/command.go
  • Line 16: warning: exported var ErrCommandNotFound should have comment or be unexported (golint)
  • Line 43: warning: comment on exported method FlagRegistryFunc.RegisterFlags should be of the form "RegisterFlags ..." (golint)
  • Line 48: warning: exported type ArgsProcessor should have comment or be unexported (golint)
  • Line 55: warning: comment on exported method ArgsProcessorFunc.ProcessArgs should be of the form "ProcessArgs ..." (golint)
  • Line 60: warning: exported type Command should have comment or be unexported (golint)
  • Line 66: warning: exported type Params should have comment or be unexported (golint)
  • Line 76: warning: exported function New should have comment or be unexported (golint)
  • Line 97: warning: exported method Command.SetWriter should have comment or be unexported (golint)
  • Line 105: warning: exported method Command.Name should have comment or be unexported (golint)
  • Line 109: warning: exported method Command.Desc should have comment or be unexported (golint)
  • Line 113: warning: exported method Command.Usage should have comment or be unexported (golint)
  • Line 163: warning: exported method Command.Exec should have comment or be unexported (golint)

• peer-calls/server/udpmux/udpmux.go
  • Line 12: warning: exported const DefaultMTU should have comment or be unexported (golint)
  • Line 14: warning: exported type UDPMux should have comment or be unexported (golint)
  • Line 26: warning: exported type Params should have comment or be unexported (golint)
  • Line 34: warning: exported function New should have comment or be unexported (golint)
  • Line 59: warning: exported method UDPMux.LocalAddr should have comment or be unexported (golint)
  • Line 83: warning: exported method UDPMux.GetConn should have comment or be unexported (golint)
  • Line 282: warning: exported method UDPMux.Done should have comment or be unexported (golint)
  • Line 286: warning: exported method UDPMux.Close should have comment or be unexported (golint)

• peer-calls/server/room.go
  • Line 11: warning: exported type NewAdapterFunc should have comment or be unexported (golint)
  • Line 18: warning: exported type AdapterRoomManager should have comment or be unexported (golint)
  • Line 26: warning: exported function NewAdapterRoomManager should have comment or be unexported (golint)
  • Line 33: warning: exported method AdapterRoomManager.Enter should have comment or be unexported (golint)
  • Line 51: warning: exported method AdapterRoomManager.Exit should have comment or be unexported (golint)
  • Line 70: warning: exported type ChannelRoomManager should have comment or be unexported (golint)
  • Line 78: warning: exported function NewChannelRoomManager should have comment or be unexported (golint)
  • Line 105: warning: exported method ChannelRoomManager.Enter should have comment or be unexported (golint)
  • Line 119: warning: exported method ChannelRoomManager.Exit should have comment or be unexported (golint)
  • Line 133: warning: exported method ChannelRoomManager.AcceptEvent should have comment or be unexported (golint)
  • Line 142: warning: exported method ChannelRoomManager.RoomEventsChannel should have comment or be unexported (golint)
  • Line 146: warning: exported type RoomEvent should have comment or be unexported (golint)
  • Line 151: warning: exported type RoomEventType should have comment or be unexported (golint)
  • Line 154: warning: exported const RoomEventTypeAdd should have comment (or a comment on this block) or be unexported (golint)

• peer-calls/server/wsclient.go
  • Line 17: warning: exported type WSWriter should have comment or be unexported (golint)
  • Line 21: warning: exported type WSReader should have comment or be unexported (golint)
  • Line 25: warning: exported type WSCloser should have comment or be unexported (golint)
  • Line 29: warning: exported type WSReadWriter should have comment or be unexported (golint)
  • Line 35: warning: comment on exported type Client should be of the form "Client ..." (with optional leading article) (golint)
  • Line 50: warning: comment on exported function NewClient should be of the form "NewClient ..." (golint)
  • Line 55: warning: exported function NewClientWithID should have comment or be unexported (golint)
  • Line 72: warning: exported method Client.SetMetadata should have comment or be unexported (golint)
  • Line 76: warning: exported method Client.Metadata should have comment or be unexported (golint)
  • Line 80: warning: comment on exported method Client.WriteCtx should be of the form "WriteCtx ..." (golint)
  • Line 91: warning: exported method Client.ID should have comment or be unexported (golint)

• peer-calls/server/wsmessage.go
  • Line 10: warning: exported type Serializer should have comment or be unexported (golint)
  • Line 14: warning: exported type Deserializer should have comment or be unexported (golint)
  • Line 18: warning: exported type ByteSerializer should have comment or be unexported (golint)
  • Line 20: warning: exported method ByteSerializer.Serialize should have comment or be unexported (golint)
  • Line 25: warning: exported method ByteSerializer.Deserialize should have comment or be unexported (golint)

• peer-calls/server/logformatter/log_formatter.go
  • Line 15: warning: exported function New should have comment or be unexported (golint)
  • Line 21: warning: exported method LogFormatter.Format should have comment or be unexported (golint)

• peer-calls/server/logger/config.go
  • Line 44: warning: exported function NewConfig should have comment or be unexported (golint)

• peer-calls/server/pubsub/errors.go
  • Line 6: warning: exported var ErrTrackNotFound should have comment or be unexported (golint)

• peer-calls/server/pubsub/track_reader.go
  • Line 14: warning: exported type Reader should have comment or be unexported (golint)
  • Line 24: warning: exported type TrackReader should have comment or be unexported (golint)
  • Line 35: warning: exported function NewTrackReader should have comment or be unexported (golint)
  • Line 48: warning: exported method TrackReader.Track should have comment or be unexported (golint)
  • Line 84: warning: exported method TrackReader.Sub should have comment or be unexported (golint)
  • Line 126: warning: exported method TrackReader.Unsub should have comment or be unexported (golint)
  • Line 136: warning: exported method TrackReader.Subs should have comment or be unexported (golint)
  • Line 152: warning: exported method TrackReader.SSRC should have comment or be unexported (golint)
  • Line 156: warning: exported method TrackReader.RID should have comment or be unexported (golint)

• peer-calls/server/sfu/tee.go
  • Line 3: warning: exported type Reader should have comment or be unexported (golint)

• peer-calls/server/servertransport/metadata_transport.go
  • Line 27: warning: exported type MetadataTransport should have comment or be unexported (golint)
  • Line 44: warning: exported type MetadataTransportParams should have comment or be unexported (golint)
  • Line 63: warning: exported function NewMetadataTransport should have comment or be unexported (golint)
  • Line 292: warning: exported method MetadataTransport.RemoteTracksChannel should have comment or be unexported (golint)
  • Line 300: warning: exported method MetadataTransport.LocalTracks should have comment or be unexported (golint)
  • Line 315: warning: exported method MetadataTransport.AddTrack should have comment or be unexported (golint)
  • Line 374: warning: exported method MetadataTransport.RemoveTrack should have comment or be unexported (golint)
  • Line 408: warning: exported method MetadataTransport.Close should have comment or be unexported (golint)

• peer-calls/server/adapterfactory.go
  • Line 13: warning: exported type AdapterFactory should have comment or be unexported (golint)
  • Line 20: warning: exported function NewAdapterFactory should have comment or be unexported (golint)
  • Line 56: warning: exported method AdapterFactory.Close should have comment or be unexported (golint)

• peer-calls/server/addr.go
  • Line 10: warning: exported function ParseUDPAddr should have comment or be unexported (golint)
  • Line 32: warning: exported function ParseUDPAddrs should have comment or be unexported (golint)

• peer-calls/server/stringmux/packetizer.go
  • Line 8: warning: exported const StringMuxByte should have comment (or a comment on this block) or be unexported (golint)
  • Line 13: warning: exported var ErrStreamIDTooLarge should have comment or be unexported (golint)
  • Line 17: warning: exported function Marshal should have comment or be unexported (golint)
  • Line 54: warning: exported function Unmarshal should have comment or be unexported (golint)

• peer-calls/server/network_types.go
  • Line 9: warning: exported function NewNetworkTypes should have comment or be unexported (golint)

• peer-calls/server/transport/track_json.go
  • Line 5: warning: exported type TrackJSON should have comment or be unexported (golint)

• peer-calls/server/sfu/stats/receiver.go
  • Line 10: warning: exported type Receiver should have comment or be unexported (golint)
  • Line 16: warning: exported function NewReceiver should have comment or be unexported (golint)
  • Line 23: warning: exported method Receiver.ReceiveRTP should have comment or be unexported (golint)
  • Line 53: warning: exported method Receiver.ReceiveRTCP should have comment or be unexported (golint)
  • Line 64: warning: exported method Receiver.Stats should have comment or be unexported (golint)

• peer-calls/server/udpmux/conn.go
  • Line 13: warning: exported type Conn should have comment or be unexported (golint)

• peer-calls/server/identifiers/identifiers.go
  • Line 8: warning: exported type TrackID should have comment or be unexported (golint)
  • Line 15: warning: exported type RoomID should have comment or be unexported (golint)
  • Line 24: warning: exported type ClientIDs should have comment or be unexported (golint)
  • Line 48: warning: exported const ServerNodePrefix should have comment or be unexported (golint)
  • Line 50: warning: exported method ClientID.IsServer should have comment or be unexported (golint)

• peer-calls/server/transport/transport.go
  • Line 11: warning: exported type Type should have comment or be unexported (golint)
  • Line 14: warning: exported const TypeWebRTC should have comment (or a comment on this block) or be unexported (golint)
  • Line 18: warning: exported type Transport should have comment or be unexported (golint)
  • Line 36: warning: exported type Closable should have comment or be unexported (golint)
  • Line 45: warning: exported type TrackLocal should have comment or be unexported (golint)
  • Line 51: warning: exported type TrackRemote should have comment or be unexported (golint)
  • Line 58: warning: exported type RTCPReader should have comment or be unexported (golint)
  • Line 62: warning: exported type TrackRemoteWithRTCPReader should have comment or be unexported (golint)
  • Line 67: warning: exported type RTCPWriter should have comment or be unexported (golint)
  • Line 71: warning: exported type DataTransport should have comment or be unexported (golint)

• peer-calls/server/stringmux/errors.go
  • Line 6: warning: exported var ErrConnAlreadyExists should have comment or be unexported (golint)

• peer-calls/server/cli/play/rtcp_reader.go
  • Line 18: warning: exported type RTCPReaderParams should have comment or be unexported (golint)
  • Line 24: warning: exported function NewRTCPReader should have comment or be unexported (golint)
  • Line 34: warning: exported method RTCPReader.ReadRTCP should have comment or be unexported (golint)
  • Line 56: warning: exported method RTCPReader.Close should have comment or be unexported (golint)

• peer-calls/server/cli/play/rtp_reader.go
  • Line 22: warning: exported type RTPReaderParams should have comment or be unexported (golint)
  • Line 31: warning: exported function NewRTPReader should have comment or be unexported (golint)
  • Line 56: warning: exported method RTPReader.ReadRTP should have comment or be unexported (golint)
  • Line 80: warning: exported method RTPReader.Close should have comment or be unexported (golint)

• peer-calls/server/sfu/peer_manager.go
  • Line 18: warning: exported var ErrDuplicateTransport should have comment or be unexported (golint)
  • Line 20: warning: exported type PeerManager should have comment or be unexported (golint)
  • Line 38: warning: exported function NewPeerManager should have comment or be unexported (golint)
  • Line 258: warning: exported method PeerManager.Sub should have comment or be unexported (golint)
  • Line 379: warning: exported method PeerManager.Unsub should have comment or be unexported (golint)
  • Line 439: warning: exported method PeerManager.Close should have comment or be unexported (golint)

• peer-calls/server/sfu/tracks_manager.go
  • Line 13: warning: exported const DataChannelName should have comment or be unexported (golint)
  • Line 15: warning: exported type TracksManager should have comment or be unexported (golint)
  • Line 22: warning: exported function NewTracksManager should have comment or be unexported (golint)
  • Line 103: warning: exported method TracksManager.Sub should have comment or be unexported (golint)
  • Line 117: warning: exported method TracksManager.Unsub should have comment or be unexported (golint)

• peer-calls/server/servertransport/data_transport.go
  • Line 13: warning: exported type DataTransport should have comment or be unexported (golint)
  • Line 21: warning: exported type DataTransportParams should have comment or be unexported (golint)
  • Line 26: warning: exported function NewDataTransport should have comment or be unexported (golint)
  • Line 72: warning: exported method DataTransport.MessagesChannel should have comment or be unexported (golint)
  • Line 76: warning: exported method DataTransport.Send should have comment or be unexported (golint)
  • Line 97: warning: exported method DataTransport.SendText should have comment or be unexported (golint)
  • Line 108: warning: exported method DataTransport.Close should have comment or be unexported (golint)

• peer-calls/server/multierr/multierr.go
  • Line 11: warning: exported type MultiErr should have comment or be unexported (golint)
  • Line 16: warning: exported function New should have comment or be unexported (golint)
  • Line 60: warning: exported function Is should have comment or be unexported (golint)

• peer-calls/server/wrtcnegotiator.go
  • Line 11: warning: exported type TransceiverRequest should have comment or be unexported (golint)
  • Line 16: warning: exported type Negotiator should have comment or be unexported (golint)
  • Line 31: warning: exported function NewNegotiator should have comment or be unexported (golint)
  • Line 50: warning: exported method Negotiator.AddTransceiverFromKind should have comment or be unexported (golint)
  • Line 73: warning: exported method Negotiator.Done should have comment or be unexported (golint)
  • Line 111: warning: exported method Negotiator.Negotiate should have comment or be unexported (golint)

• peer-calls/server/test/unsetenvprefix.go
  • Line 8: warning: exported function UnsetEnvPrefix should have comment or be unexported (golint)

• peer-calls/server/sfu/jitter_buffer.go
  • Line 16: warning: exported function NewJitterBuffer should have comment or be unexported (golint)
  • Line 50: warning: exported method JitterBuffer.RemoveBuffer should have comment or be unexported (golint)

• peer-calls/server/configread.go
  • Line 13: warning: exported function ReadConfigFile should have comment or be unexported (golint)
  • Line 26: warning: exported function ReadConfigFiles should have comment or be unexported (golint)
  • Line 37: warning: exported function InitConfig should have comment or be unexported (golint)
  • Line 48: warning: exported function ReadConfig should have comment or be unexported (golint)
  • Line 56: warning: exported function ReadConfigYAML should have comment or be unexported (golint)
  • Line 65: warning: exported function ReadConfigFromEnv should have comment or be unexported (golint)

• peer-calls/server/configtypes.go
  • Line 3: warning: exported type AuthType should have comment or be unexported (golint)
  • Line 6: warning: exported const AuthTypeSecret should have comment (or a comment on this block) or be unexported (golint)
  • Line 10: warning: exported type ICEServer should have comment or be unexported (golint)
  • Line 19: warning: exported type TLSConfig should have comment or be unexported (golint)
  • Line 24: warning: exported type StoreType should have comment or be unexported (golint)
  • Line 27: warning: exported const StoreTypeMemory should have comment (or a comment on this block) or be unexported (golint)
  • Line 31: warning: exported type RedisConfig should have comment or be unexported (golint)
  • Line 37: warning: exported type StoreConfig should have comment or be unexported (golint)
  • Line 42: warning: exported type NetworkType should have comment or be unexported (golint)
  • Line 45: warning: exported const NetworkTypeMesh should have comment (or a comment on this block) or be unexported (golint)
  • Line 49: warning: exported type NetworkConfig should have comment or be unexported (golint)
  • Line 54: warning: exported type NetworkConfigSFU should have comment or be unexported (golint)
  • Line 68: warning: exported type TransportConfig should have comment or be unexported (golint)
  • Line 73: warning: exported type PrometheusConfig should have comment or be unexported (golint)
  • Line 77: warning: exported type Config should have comment or be unexported (golint)
  • Line 90: warning: exported type ICEAuthServer should have comment or be unexported (golint)
  • Line 96: warning: exported type ClientConfig should have comment or be unexported (golint)

• peer-calls/server/datatransceiver.go
  • Line 13: warning: exported type DataTransceiver should have comment or be unexported (golint)
  • Line 42: warning: exported function NewDataTransceiver should have comment or be unexported (golint)
  • Line 138: warning: exported method DataTransceiver.MessagesChannel should have comment or be unexported (golint)
  • Line 142: warning: exported method DataTransceiver.Close should have comment or be unexported (golint)
  • Line 153: warning: exported method DataTransceiver.Send should have comment or be unexported (golint)

• peer-calls/server/roommanagerfactory.go
  • Line 8: warning: exported type RoomManagerFactory should have comment or be unexported (golint)
  • Line 12: warning: exported type RoomManagerFactoryParams should have comment or be unexported (golint)
  • Line 18: warning: exported function NewRoomManagerFactory should have comment or be unexported (golint)
  • Line 26: warning: exported method RoomManagerFactory.NewRoomManager should have comment or be unexported (golint)

• peer-calls/server/wsadapter.go
  • Line 9: warning: exported type ClientWriter should have comment or be unexported (golint)
  • Line 16: warning: exported type Adapter should have comment or be unexported (golint)
  • Line 28: warning: exported var ErrDuplicateClientID should have comment or be unexported (golint)

• peer-calls/server/transport/codec.go
  • Line 9: warning: exported type Codec should have comment or be unexported (golint)
  • Line 16: warning: exported method Codec.TrackKind should have comment or be unexported (golint)
  • Line 24: warning: exported type TrackKind should have comment or be unexported (golint)
  • Line 27: warning: exported const TrackKindAudio should have comment (or a comment on this block) or be unexported (golint)
  • Line 31: warning: exported function NewTrackKind should have comment or be unexported (golint)
  • Line 39: warning: exported method TrackKind.RTPCodecType should have comment or be unexported (golint)

• peer-calls/server/memoryadapter.go
  • Line 11: warning: exported type MemoryAdapter should have comment or be unexported (golint)
  • Line 17: warning: exported function NewMemoryAdapter should have comment or be unexported (golint)
  • Line 54: warning: exported method MemoryAdapter.Close should have comment or be unexported (golint)
  • Line 67: warning: exported method MemoryAdapter.Metadata should have comment or be unexported (golint)
  • Line 79: warning: exported method MemoryAdapter.SetMetadata should have comment or be unexported (golint)
  • Line 91: warning: comment on exported method MemoryAdapter.Clients should be of the form "Clients ..." (golint)
  • Line 105: warning: exported method MemoryAdapter.Size should have comment or be unexported (golint)
  • Line 112: warning: comment on exported method MemoryAdapter.Broadcast should be of the form "Broadcast ..." (golint)
  • Line 132: warning: comment on exported method MemoryAdapter.Emit should be of the form "Emit ..." (golint)

• peer-calls/server/nodemanager.go
  • Line 20: warning: exported type NodeManager should have comment or be unexported (golint)
  • Line 27: warning: exported type NodeManagerParams should have comment or be unexported (golint)
  • Line 35: warning: exported function NewNodeManager should have comment or be unexported (golint)
  • Line 268: warning: exported method NodeManager.Close should have comment or be unexported (golint)

• peer-calls/server/redisadapter.go
  • Line 23: warning: exported type JSONMessage should have comment or be unexported (golint)
  • Line 28: warning: exported type Doner should have comment or be unexported (golint)
  • Line 32: warning: exported type RedisAdapter should have comment or be unexported (golint)
  • Line 68: warning: exported function NewRedisAdapter should have comment or be unexported (golint)
  • Line 104: warning: exported method RedisAdapter.Add should have comment or be unexported (golint)
  • Line 142: warning: exported method RedisAdapter.Remove should have comment or be unexported (golint)
  • Line 187: warning: exported method RedisAdapter.Metadata should have comment or be unexported (golint)
  • Line 193: warning: exported method RedisAdapter.SetMetadata should have comment or be unexported (golint)
  • Line 210: warning: comment on exported method RedisAdapter.Clients should be of the form "Clients ..." (golint)
  • Line 230: warning: comment on exported method RedisAdapter.Size should be of the form "Size ..." (golint)
  • Line 458: warning: exported method RedisAdapter.Broadcast should have comment or be unexported (golint)
  • Line 487: warning: exported method RedisAdapter.Emit should have comment or be unexported (golint)

• peer-calls/server/webrtctransport.go
  • Line 20: warning: exported type WebRTCTransportFactory should have comment or be unexported (golint)
  • Line 27: warning: exported function NewWebRTCTransportFactory should have comment or be unexported (golint)
  • Line 107: warning: exported function NewMediaEngine should have comment or be unexported (golint)
  • Line 117: warning: exported function NewInterceptorRegistry should have comment or be unexported (golint)
  • Line 127: warning: exported function RegisterCodecs should have comment or be unexported (golint)
  • Line 169: warning: exported type WebRTCTransport should have comment or be unexported (golint)
  • Line 189: warning: exported method WebRTCTransportFactory.NewWebRTCTransport should have comment or be unexported (golint)
  • Line 243: warning: exported function NewWebRTCTransport should have comment or be unexported (golint)
  • Line 337: warning: exported method WebRTCTransport.Close should have comment or be unexported (golint)
  • Line 341: warning: exported method WebRTCTransport.ClientID should have comment or be unexported (golint)
  • Line 345: warning: exported method WebRTCTransport.Type should have comment or be unexported (golint)
  • Line 349: warning: exported method WebRTCTransport.WriteRTCP should have comment or be unexported (golint)
  • Line 362: warning: exported method WebRTCTransport.Done should have comment or be unexported (golint)
  • Line 366: warning: exported method WebRTCTransport.RemoteTracksChannel should have comment or be unexported (golint)
  • Line 370: warning: exported method WebRTCTransport.RemoveTrack should have comment or be unexported (golint)
  • Line 398: warning: exported method WebRTCTransport.AddTrack should have comment or be unexported (golint)
  • Line 511: warning: exported method WebRTCTransport.Signal should have comment or be unexported (golint)
  • Line 517: warning: exported method WebRTCTransport.SignalChannel should have comment or be unexported (golint)
  • Line 521: warning: exported method WebRTCTransport.MessagesChannel should have comment or be unexported (golint)
  • Line 525: warning: exported method WebRTCTransport.Send should have comment or be unexported (golint)
  • Line 529: warning: exported type LocalTrack should have comment or be unexported (golint)
  • Line 534: warning: exported method LocalTrack.Track should have comment or be unexported (golint)
  • Line 538: warning: exported type RemoteTrack should have comment or be unexported (golint)
  • Line 543: warning: exported method RemoteTrack.Track should have comment or be unexported (golint)

• peer-calls/server/servertransport/match.go
  • Line 53: warning: comment on exported function MatchRTCP should be of the form "MatchRTCP ..." (golint)

• peer-calls/server/server.go
  • Line 12: warning: exported type Params should have comment or be unexported (golint)
  • Line 17: warning: exported type Server should have comment or be unexported (golint)
  • Line 22: warning: exported function New should have comment or be unexported (golint)
  • Line 32: warning: exported method Server.Start should have comment or be unexported (golint)

• peer-calls/server/logger/level.go
  • Line 34: warning: exported const LevelDisabledString should have comment (or a comment on this block) or be unexported (golint)
  • Line 64: warning: exported function LevelFromString should have comment or be unexported (golint)

• peer-calls/server/pubsub/pub_track_event.go
  • Line 7: warning: exported type PubTrackEvent should have comment or be unexported (golint)

• peer-calls/server/test/testcloser.go
  • Line 5: warning: comment on exported type Closer should be of the form "Closer ..." (with optional leading article) (golint)
  • Line 10: warning: exported method Closer.Close should have comment or be unexported (golint)
  • Line 25: warning: exported method Closer.Add should have comment or be unexported (golint)
  • Line 33: warning: exported method Closer.AddFuncErr should have comment or be unexported (golint)

• peer-calls/server/test/timeout.go
  • Line 13: warning: exported function Timeout should have comment or be unexported (golint)

• peer-calls/server/sfu/buffer.go
  • Line 56: warning: exported method Buffer.Push should have comment or be unexported (golint)
  • Line 98: warning: exported method Buffer.GetPacket should have comment or be unexported (golint)
  • Line 102: warning: exported method Buffer.SSRC should have comment or be unexported (golint)
  • Line 210: warning: exported function AddBLP should have comment or be unexported (golint)
  • Line 214: warning: exported function SubBLP should have comment or be unexported (golint)
  • Line 218: warning: exported function CreateNackPair should have comment or be unexported (golint)

• peer-calls/server/embed.go
  • Line 5: warning: exported type Embed should have comment or be unexported (golint)

• peer-calls/server/errors.go
  • Line 10: warning: exported type MultiErrorHandler should have comment or be unexported (golint)

• peer-calls/server/message/signal.go
  • Line 8: warning: exported type Signal should have comment or be unexported (golint)
  • Line 16: warning: exported type SignalType should have comment or be unexported (golint)
  • Line 19: warning: exported const SignalTypeCandidate should have comment (or a comment on this block) or be unexported (golint)
  • Line 28: warning: exported function NewSignalTypeFromSDPType should have comment or be unexported (golint)
  • Line 43: warning: exported method SignalType.SDPType should have comment or be unexported (golint)
  • Line 58: warning: exported type TransceiverRequest should have comment or be unexported (golint)
  • Line 63: warning: exported type TransceiverInit should have comment or be unexported (golint)
  • Line 67: warning: exported type Direction should have comment or be unexported (golint)
  • Line 70: warning: exported const DirectionSendRecv should have comment (or a comment on this block) or be unexported (golint)
  • Line 76: warning: exported method Direction.RTPTransceiverDirection should have comment or be unexported (golint)

• peer-calls/server/stringmux/conn.go
  • Line 13: warning: exported type Conn should have comment or be unexported (golint)

• peer-calls/server/transport/track.go
  • Line 7: warning: exported type Track should have comment or be unexported (golint)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

misspell99%

Misspell Finds commonly misspelled English words

• peer-calls/server/sfu/stats/timer.go
  • Line 22: warning: "respones" is a misspelling of "response" (misspell)