Preparing report...

Report for github.com/ermites-io/oidc

A+    Excellent!    Found 15 issues across 20 files

Tweet

gofmt100%

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

No problems detected. Good job!


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!


gocyclo100%

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.

No problems detected. Good job!


golint25%

Golint is a linter for Go source code.

    • oidc/internal/state/state.go
    • Line 19: warning: exported const MaxUserDataSize should have comment (or a comment on this block) or be unexported (golint)
    • Line 38: warning: comment on exported function NewData should be of the form "NewData ..." (golint)
    • Line 52: warning: comment on exported method Data.Pack should be of the form "Pack ..." (golint)
    • Line 57: warning: comment on exported function ParseData should be of the form "ParseData ..." (golint)
    • Line 64: warning: comment on exported function NewEnvelope should be of the form "NewEnvelope ..." (golint)
    • Line 91: warning: exported method Envelope.Seal should have comment or be unexported (golint)
    • Line 121: warning: exported method Envelope.Open should have comment or be unexported (golint)
    • Line 148: warning: exported method Envelope.Pack should have comment or be unexported (golint)
    • Line 162: warning: exported function ParseEnvelope should have comment or be unexported (golint)
    • oidc/internal/token/id.go
    • Line 11: warning: exported type Id should have comment or be unexported (golint)
    • Line 25: warning: exported method Id.GetHeader should have comment or be unexported (golint)
    • Line 29: warning: exported method Id.GetClaims should have comment or be unexported (golint)
    • Line 33: warning: exported method Id.GetSignature should have comment or be unexported (golint)
    • Line 42: warning: exported method Id.GetVerifyInfo should have comment or be unexported (golint)
    • Line 49: warning: comment on exported function Parse should be of the form "Parse ..." (golint)
    • oidc/const.go
    • Line 5: warning: exported type Error should have comment or be unexported (golint)
    • Line 10: warning: exported const ErrParse should have comment (or a comment on this block) or be unexported (golint)
    • oidc/var.go
    • Line 8: warning: exported var DefaultStateTimeout should have comment or be unexported (golint)
    • oidc/internal/jwk/jwk.go
    • Line 21: warning: exported type Keys should have comment or be unexported (golint)
    • Line 23: warning: exported method Keys.Verify should have comment or be unexported (golint)
    • Line 84: warning: exported function MapFromUrl should have comment or be unexported (golint)
    • oidc/internal/token/errors.go
    • Line 5: warning: exported type Error should have comment or be unexported (golint)
    • Line 10: warning: exported const ErrParse should have comment (or a comment on this block) or be unexported (golint)
    • oidc/internal/token/id_header.go
    • Line 20: warning: comment on exported type Header should be of the form "Header ..." (with optional leading article) (golint)
    • Line 46: warning: exported method Header.GetKid should have comment or be unexported (golint)
    • Line 50: warning: exported method Header.GetAlg should have comment or be unexported (golint)
    • Line 54: warning: exported method Header.GetRaw should have comment or be unexported (golint)
    • Line 72: warning: exported function ParseHeader should have comment or be unexported (golint)
    • oidc/openid.go
    • Line 11: warning: exported type OpenIDConfiguration should have comment or be unexported (golint)
    • oidc/internal/jwk/errors.go
    • Line 5: warning: exported type Error should have comment or be unexported (golint)
    • Line 10: warning: exported const ErrParse should have comment (or a comment on this block) or be unexported (golint)
    • oidc/internal/token/id_claims.go
    • Line 11: warning: comment on exported type Claims should be of the form "Claims ..." (with optional leading article) (golint)
    • Line 40: warning: exported method Claims.GetRaw should have comment or be unexported (golint)
    • Line 40: warning: receiver name c should be consistent with previous receiver name h for Claims (golint)
    • Line 45: warning: receiver name c should be consistent with previous receiver name h for Claims (golint)
    • Line 58: warning: exported function ParseClaims should have comment or be unexported (golint)
    • oidc/internal/token/id_sig.go
    • Line 10: warning: comment on exported type Signature should be of the form "Signature ..." (with optional leading article) (golint)
    • Line 23: warning: exported method Signature.GetBlob should have comment or be unexported (golint)
    • Line 27: warning: exported function ParseSignature should have comment or be unexported (golint)
    • oidc/internal/token/token.go
    • Line 10: warning: exported type Response should have comment or be unexported (golint)
    • Line 19: warning: exported type ErrorMessage should have comment or be unexported (golint)
    • Line 34: warning: comment on exported method Response.Valid should be of the form "Valid ..." (golint)
    • Line 54: warning: exported function ParseResponse should have comment or be unexported (golint)
    • oidc/internal/state/auth.go
    • Line 16: warning: exported type Verifier should have comment or be unexported (golint)
    • Line 21: warning: exported function NewVerifier should have comment or be unexported (golint)
    • Line 110: warning: exported method Verifier.New should have comment or be unexported (golint)
    • Line 114: warning: comment on exported method Verifier.NewWithData should be of the form "NewWithData ..." (golint)
    • Line 157: warning: exported method Verifier.Validate should have comment or be unexported (golint)
    • Line 162: warning: exported method Verifier.ValidateWithData should have comment or be unexported (golint)
    • oidc/internal/state/errors.go
    • Line 5: warning: exported type Error should have comment or be unexported (golint)
    • Line 10: warning: exported const ErrParse should have comment (or a comment on this block) or be unexported (golint)
    • oidc/provider.go
    • Line 18: warning: exported type Provider should have comment or be unexported (golint)
    • Line 38: warning: exported type Token should have comment or be unexported (golint)
    • Line 45: warning: exported function NewProvider should have comment or be unexported (golint)
    • Line 65: warning: exported function NewProviderOauthOnly should have comment or be unexported (golint)
    • Line 76: warning: exported method Provider.SetAuth should have comment or be unexported (golint)
    • Line 103: warning: exported method Provider.RequestIdentityParams should have comment or be unexported (golint)
    • Line 107: warning: comment on exported method Provider.RequestIdentityParamsWithUserdata should be of the form "RequestIdentityParamsWithUserdata ..." (golint)
    • Line 155: warning: comment on exported method Provider.ValidateIdentityParams should be of the form "ValidateIdentityParams ..." (golint)
    • Line 161: warning: exported method Provider.ValidateIdentityParamsWithUserdata should have comment or be unexported (golint)
    • Line 219: warning: exported method Provider.GetName should have comment or be unexported (golint)
    • Line 223: warning: comment on exported function GetProvider should be of the form "GetProvider ..." (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!