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!

gocyclo93%

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.

• oathkeeper/pipeline/errors/when.go
  • Line 145: warning: cyclomatic complexity 18 of function matchesRequest() is high (> 15) (gocyclo)

• oathkeeper/driver/configuration/provider_viper.go
  • Line 298: warning: cyclomatic complexity 18 of function (*ViperProvider).PipelineConfig() is high (> 15) (gocyclo)

• oathkeeper/credentials/fetcher_default.go
  • Line 210: warning: cyclomatic complexity 16 of function (*FetcherDefault).resolve() is high (> 15) (gocyclo)

• oathkeeper/credentials/signer_default.go
  • Line 48: warning: cyclomatic complexity 16 of function (*DefaultSigner).key() is high (> 15) (gocyclo)

• oathkeeper/rule/fetcher_default.go
  • Line 192: warning: cyclomatic complexity 21 of function (*FetcherDefault).watch() is high (> 15) (gocyclo)

• oathkeeper/pipeline/authz/keto_engine_acp_ory.go
  • Line 107: warning: cyclomatic complexity 17 of function (*AuthorizerKetoEngineACPORY).Authorize() is high (> 15) (gocyclo)

• oathkeeper/proxy/request_handler.go
  • Line 170: warning: cyclomatic complexity 18 of function (*RequestHandler).HandleRequest() is high (> 15) (gocyclo)
  • Line 86: warning: cyclomatic complexity 16 of function (*RequestHandler).HandleError() is high (> 15) (gocyclo)

• oathkeeper/credentials/verifier_default.go
  • Line 32: warning: cyclomatic complexity 29 of function (*VerifierDefault).Verify() is high (> 15) (gocyclo)

• oathkeeper/pipeline/mutate/mutator_hydrator.go
  • Line 144: warning: cyclomatic complexity 23 of function (*MutatorHydrator).Mutate() is high (> 15) (gocyclo)

• oathkeeper/rule/rule_migrator.go
  • Line 19: warning: cyclomatic complexity 22 of function migrateRuleJSON() is high (> 15) (gocyclo)

• oathkeeper/pipeline/authn/authenticator_oauth2_introspection.go
  • Line 152: warning: cyclomatic complexity 22 of function (*AuthenticatorOAuth2Introspection).Authenticate() is high (> 15) (gocyclo)

golint54%

Golint is a linter for Go source code.

• oathkeeper/proxy/response_writer.go
  • Line 34: warning: exported function NewSimpleResponseWriter should have comment or be unexported (golint)

• oathkeeper/pipeline/errors/error_redirect.go
  • Line 16: warning: exported type ErrorRedirectConfig should have comment or be unexported (golint)
  • Line 21: warning: exported type ErrorRedirect should have comment or be unexported (golint)
  • Line 25: warning: exported type ErrorRedirectDependencies should have comment or be unexported (golint)
  • Line 30: warning: exported function NewErrorRedirect should have comment or be unexported (golint)
  • Line 37: warning: exported method ErrorRedirect.Handle should have comment or be unexported (golint)
  • Line 47: warning: exported method ErrorRedirect.Validate should have comment or be unexported (golint)
  • Line 55: warning: exported method ErrorRedirect.Config should have comment or be unexported (golint)
  • Line 68: warning: exported method ErrorRedirect.GetID should have comment or be unexported (golint)
  • Line 72: warning: exported method ErrorRedirect.RedirectURL should have comment or be unexported (golint)

• oathkeeper/driver/health/event_manager.go
  • Line 11: warning: exported type EventManager should have comment or be unexported (golint)
  • Line 17: warning: exported var ErrEventTypeAlreadyRegistered should have comment or be unexported (golint)

• oathkeeper/api/rule.go
  • Line 37: warning: exported const RulesPath should have comment (or a comment on this block) or be unexported (golint)
  • Line 40: warning: exported type RuleHandler should have comment or be unexported (golint)
  • Line 49: warning: exported function NewRuleHandler should have comment or be unexported (golint)
  • Line 53: warning: exported method RuleHandler.SetRoutes should have comment or be unexported (golint)

• oathkeeper/pipeline/mutate/mutator_cookie.go
  • Line 20: warning: exported type CredentialsCookiesConfig should have comment or be unexported (golint)
  • Line 24: warning: exported type MutatorCookie should have comment or be unexported (golint)
  • Line 29: warning: exported function NewMutatorCookie should have comment or be unexported (golint)
  • Line 33: warning: exported method MutatorCookie.GetID should have comment or be unexported (golint)
  • Line 37: warning: exported method MutatorCookie.WithCache should have comment or be unexported (golint)
  • Line 41: warning: exported method MutatorCookie.Mutate should have comment or be unexported (golint)
  • Line 96: warning: exported method MutatorCookie.Validate should have comment or be unexported (golint)

• oathkeeper/pipeline/authn/authenticator_oauth2_introspection.go
  • Line 31: warning: exported type AuthenticatorOAuth2IntrospectionConfiguration should have comment or be unexported (golint)
  • Line 44: warning: exported type AuthenticatorOAuth2IntrospectionPreAuthConfiguration should have comment or be unexported (golint)
  • Line 53: warning: exported type AuthenticatorOAuth2IntrospectionRetryConfiguration should have comment or be unexported (golint)
  • Line 64: warning: exported type AuthenticatorOAuth2Introspection should have comment or be unexported (golint)
  • Line 75: warning: exported function NewAuthenticatorOAuth2Introspection should have comment or be unexported (golint)
  • Line 79: warning: exported method AuthenticatorOAuth2Introspection.GetID should have comment or be unexported (golint)
  • Line 83: warning: exported type AuthenticatorOAuth2IntrospectionResult should have comment or be unexported (golint)
  • Line 152: warning: exported method AuthenticatorOAuth2Introspection.Authenticate should have comment or be unexported (golint)
  • Line 252: warning: exported method AuthenticatorOAuth2Introspection.Validate should have comment or be unexported (golint)
  • Line 261: warning: exported method AuthenticatorOAuth2Introspection.Config should have comment or be unexported (golint)

• oathkeeper/rule/rule.go
  • Line 34: warning: exported type Match should have comment or be unexported (golint)
  • Line 55: warning: exported type Handler should have comment or be unexported (golint)
  • Line 65: warning: exported type ErrorHandler should have comment or be unexported (golint)
  • Line 74: warning: exported type OnErrorRequest should have comment or be unexported (golint)
  • Line 127: warning: exported type Upstream should have comment or be unexported (golint)
  • Line 141: warning: exported method Rule.UnmarshalJSON should have comment or be unexported (golint)

• oathkeeper/credentials/verifier_default.go
  • Line 22: warning: exported type VerifierDefault should have comment or be unexported (golint)
  • Line 26: warning: exported function NewVerifierDefault should have comment or be unexported (golint)
  • Line 32: warning: exported method VerifierDefault.Verify should have comment or be unexported (golint)

• oathkeeper/rule/fetcher.go
  • Line 25: warning: exported type Fetcher should have comment or be unexported (golint)

• oathkeeper/rule/fetcher_default.go
  • Line 37: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 63: warning: exported type FetcherDefault should have comment or be unexported (golint)
  • Line 78: warning: exported function NewFetcherDefault should have comment or be unexported (golint)
  • Line 175: warning: exported method FetcherDefault.Watch should have comment or be unexported (golint)

• oathkeeper/driver/driver_default.go
  • Line 9: warning: exported type DefaultDriver should have comment or be unexported (golint)
  • Line 14: warning: exported function NewDefaultDriver should have comment or be unexported (golint)
  • Line 21: warning: exported method DefaultDriver.Configuration should have comment or be unexported (golint)
  • Line 25: warning: exported method DefaultDriver.Registry should have comment or be unexported (golint)

• oathkeeper/driver/registry.go
  • Line 23: warning: exported type Registry should have comment or be unexported (golint)
  • Line 57: warning: exported function NewRegistry should have comment or be unexported (golint)

• oathkeeper/cmd/server/server.go
  • Line 183: warning: exported function RunServe should have comment or be unexported (golint)

• oathkeeper/proxy/request_handler.go
  • Line 54: warning: exported type RequestHandler should have comment or be unexported (golint)
  • Line 63: warning: exported function NewRequestHandler should have comment or be unexported (golint)
  • Line 86: warning: exported method RequestHandler.HandleError should have comment or be unexported (golint)
  • Line 170: warning: exported method RequestHandler.HandleRequest should have comment or be unexported (golint)

• oathkeeper/x/router.go
  • Line 9: warning: exported type RouterAPI should have comment or be unexported (golint)
  • Line 13: warning: exported function NewAPIRouter should have comment or be unexported (golint)

• oathkeeper/metrics/prometheus.go
  • Line 1: warning: package comment should be of the form "Package metrics ..." (golint)

• oathkeeper/pipeline/authz/authorizer_allow.go
  • Line 32: warning: exported type AuthorizerAllow should have comment or be unexported (golint)
  • Line 36: warning: exported function NewAuthorizerAllow should have comment or be unexported (golint)
  • Line 40: warning: exported method AuthorizerAllow.GetID should have comment or be unexported (golint)
  • Line 44: warning: exported method AuthorizerAllow.Authorize should have comment or be unexported (golint)
  • Line 48: warning: exported method AuthorizerAllow.Validate should have comment or be unexported (golint)

• oathkeeper/pipeline/authn/authenticator_noop.go
  • Line 11: warning: exported type AuthenticatorNoOp should have comment or be unexported (golint)
  • Line 15: warning: exported function NewAuthenticatorNoOp should have comment or be unexported (golint)
  • Line 19: warning: exported method AuthenticatorNoOp.GetID should have comment or be unexported (golint)
  • Line 23: warning: exported method AuthenticatorNoOp.Validate should have comment or be unexported (golint)
  • Line 34: warning: exported method AuthenticatorNoOp.Authenticate should have comment or be unexported (golint)

• oathkeeper/pipeline/context.go
  • Line 8: warning: exported type Context should have comment or be unexported (golint)
  • Line 11: warning: exported type ContextRequest should have comment or be unexported (golint)
  • Line 18: warning: exported type ContextResponse should have comment or be unexported (golint)
  • Line 28: warning: exported type AuthenticationSession should have comment or be unexported (golint)
  • Line 35: warning: exported type MatchContext should have comment or be unexported (golint)

• oathkeeper/internal/cloudstorage/setup.go
  • Line 34: warning: exported function SetCurrentTest should have comment or be unexported (golint)
  • Line 38: warning: exported function NewURLMux should have comment or be unexported (golint)
  • Line 41: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 141: warning: error should be the last type when returning multiple items (golint)
  • Line 184: warning: error should be the last type when returning multiple items (golint)
  • Line 207: warning: error should be the last type when returning multiple items (golint)
  • Line 260: warning: error should be the last type when returning multiple items (golint)

• oathkeeper/pipeline/authn/registry.go
  • Line 3: warning: exported type Registry should have comment or be unexported (golint)

• oathkeeper/pipeline/errors/error.go
  • Line 14: warning: exported type Handler should have comment or be unexported (golint)
  • Line 20: warning: exported var ErrErrorHandlerNotEnabled should have comment or be unexported (golint)
  • Line 26: warning: exported var ErrHandlerNotResponsible should have comment or be unexported (golint)
  • Line 28: warning: exported function NewErrErrorHandlerNotEnabled should have comment or be unexported (golint)
  • Line 32: warning: exported function NewErrErrorHandlerMisconfigured should have comment or be unexported (golint)

• oathkeeper/rule/matcher.go
  • Line 8: warning: exported type Matcher should have comment or be unexported (golint)

• oathkeeper/internal/driver.go
  • Line 11: warning: exported function ResetViper should have comment or be unexported (golint)
  • Line 19: warning: exported function NewConfigurationWithDefaults should have comment or be unexported (golint)
  • Line 24: warning: exported function NewRegistry should have comment or be unexported (golint)

• oathkeeper/pipeline/mutate/mutator_header.go
  • Line 18: warning: exported type MutatorHeaderConfig should have comment or be unexported (golint)
  • Line 22: warning: exported type MutatorHeader should have comment or be unexported (golint)
  • Line 27: warning: exported function NewMutatorHeader should have comment or be unexported (golint)
  • Line 31: warning: exported method MutatorHeader.GetID should have comment or be unexported (golint)
  • Line 35: warning: exported method MutatorHeader.WithCache should have comment or be unexported (golint)
  • Line 39: warning: exported method MutatorHeader.Mutate should have comment or be unexported (golint)
  • Line 69: warning: exported method MutatorHeader.Validate should have comment or be unexported (golint)

• oathkeeper/proxy/proxy.go
  • Line 46: warning: exported function NewProxy should have comment or be unexported (golint)
  • Line 50: warning: exported type Proxy should have comment or be unexported (golint)
  • Line 58: warning: exported const ContextKeyMatchedRule should have comment (or a comment on this block) or be unexported (golint)
  • Line 62: warning: exported method Proxy.RoundTrip should have comment or be unexported (golint)
  • Line 125: warning: exported method Proxy.Director should have comment or be unexported (golint)
  • Line 164: warning: exported function ConfigureBackendURL should have comment or be unexported (golint)

• oathkeeper/pipeline/authz/registry.go
  • Line 3: warning: exported type Registry should have comment or be unexported (golint)

• oathkeeper/credentials/signer.go
  • Line 10: warning: exported type Signer should have comment or be unexported (golint)
  • Line 14: warning: exported type SignerRegistry should have comment or be unexported (golint)

• oathkeeper/credentials/fetcher.go
  • Line 10: warning: exported type Fetcher should have comment or be unexported (golint)
  • Line 15: warning: exported type FetcherRegistry should have comment or be unexported (golint)

• oathkeeper/driver/driver.go
  • Line 7: warning: exported type Driver should have comment or be unexported (golint)

• oathkeeper/driver/health/event_manager_default.go
  • Line 11: warning: exported type DefaultHealthEventManager should have comment or be unexported (golint)
  • Line 17: warning: exported function NewDefaultHealthEventManager should have comment or be unexported (golint)
  • Line 34: warning: exported method DefaultHealthEventManager.Dispatch should have comment or be unexported (golint)
  • Line 43: warning: exported method DefaultHealthEventManager.Watch should have comment or be unexported (golint)
  • Line 61: warning: exported method DefaultHealthEventManager.HealthxReadyCheckers should have comment or be unexported (golint)

• oathkeeper/helper/errors.go
  • Line 30: warning: exported var ErrForbidden should have comment or be unexported (golint)
  • Line 50: warning: comment on exported var ErrNonRegexpMatchingStrategy should be of the form "ErrNonRegexpMatchingStrategy ..." (golint)

• oathkeeper/pipeline/authn/authenticator_anonymous.go
  • Line 15: warning: exported type AuthenticatorAnonymous should have comment or be unexported (golint)
  • Line 19: warning: exported type AuthenticatorAnonymousConfiguration should have comment or be unexported (golint)
  • Line 23: warning: exported function NewAuthenticatorAnonymous should have comment or be unexported (golint)
  • Line 29: warning: exported method AuthenticatorAnonymous.Validate should have comment or be unexported (golint)
  • Line 38: warning: exported method AuthenticatorAnonymous.GetID should have comment or be unexported (golint)
  • Line 42: warning: exported method AuthenticatorAnonymous.Config should have comment or be unexported (golint)
  • Line 51: warning: exported method AuthenticatorAnonymous.Authenticate should have comment or be unexported (golint)

• oathkeeper/pipeline/errors/error_json.go
  • Line 18: warning: exported type ErrorJSONConfig should have comment or be unexported (golint)
  • Line 21: warning: exported type ErrorJSON should have comment or be unexported (golint)
  • Line 30: warning: exported function NewErrorJSON should have comment or be unexported (golint)
  • Line 37: warning: exported method ErrorJSON.Handle should have comment or be unexported (golint)
  • Line 70: warning: exported method ErrorJSON.Validate should have comment or be unexported (golint)
  • Line 79: warning: exported method ErrorJSON.Config should have comment or be unexported (golint)
  • Line 88: warning: exported method ErrorJSON.GetID should have comment or be unexported (golint)

• oathkeeper/driver/registry_memory.go
  • Line 34: warning: exported type RegistryMemory should have comment or be unexported (golint)
  • Line 70: warning: exported method RegistryMemory.Init should have comment or be unexported (golint)
  • Line 80: warning: exported method RegistryMemory.RuleFetcher should have comment or be unexported (golint)
  • Line 87: warning: exported method RegistryMemory.WithRuleFetcher should have comment or be unexported (golint)
  • Line 92: warning: exported method RegistryMemory.ProxyRequestHandler should have comment or be unexported (golint)
  • Line 99: warning: exported method RegistryMemory.RuleMatcher should have comment or be unexported (golint)
  • Line 104: warning: exported function NewRegistryMemory should have comment or be unexported (golint)
  • Line 108: warning: exported method RegistryMemory.BuildVersion should have comment or be unexported (golint)
  • Line 112: warning: exported method RegistryMemory.BuildDate should have comment or be unexported (golint)
  • Line 116: warning: exported method RegistryMemory.BuildHash should have comment or be unexported (golint)
  • Line 120: warning: exported method RegistryMemory.WithConfig should have comment or be unexported (golint)
  • Line 125: warning: exported method RegistryMemory.WithBuildInfo should have comment or be unexported (golint)
  • Line 132: warning: exported method RegistryMemory.WithLogger should have comment or be unexported (golint)
  • Line 137: warning: exported method RegistryMemory.CredentialHandler should have comment or be unexported (golint)
  • Line 145: warning: exported method RegistryMemory.HealthEventManager should have comment or be unexported (golint)
  • Line 156: warning: exported method RegistryMemory.HealthHandler should have comment or be unexported (golint)
  • Line 166: warning: exported method RegistryMemory.RuleValidator should have comment or be unexported (golint)
  • Line 173: warning: exported method RegistryMemory.RuleRepository should have comment or be unexported (golint)
  • Line 180: warning: exported method RegistryMemory.Writer should have comment or be unexported (golint)
  • Line 187: warning: exported method RegistryMemory.Logger should have comment or be unexported (golint)
  • Line 194: warning: exported method RegistryMemory.RuleHandler should have comment or be unexported (golint)
  • Line 201: warning: exported method RegistryMemory.DecisionHandler should have comment or be unexported (golint)
  • Line 208: warning: exported method RegistryMemory.CredentialsFetcher should have comment or be unexported (golint)
  • Line 216: warning: exported method RegistryMemory.CredentialsSigner should have comment or be unexported (golint)
  • Line 224: warning: exported method RegistryMemory.CredentialsVerifier should have comment or be unexported (golint)
  • Line 232: warning: exported method RegistryMemory.AvailablePipelineErrorHandlers should have comment or be unexported (golint)
  • Line 245: warning: exported method RegistryMemory.PipelineErrorHandler should have comment or be unexported (golint)
  • Line 275: warning: exported method RegistryMemory.AvailablePipelineAuthenticators should have comment or be unexported (golint)
  • Line 287: warning: exported method RegistryMemory.PipelineAuthenticator should have comment or be unexported (golint)
  • Line 299: warning: exported method RegistryMemory.AvailablePipelineAuthorizers should have comment or be unexported (golint)
  • Line 311: warning: exported method RegistryMemory.PipelineAuthorizer should have comment or be unexported (golint)
  • Line 323: warning: exported method RegistryMemory.AvailablePipelineMutators should have comment or be unexported (golint)
  • Line 336: warning: exported method RegistryMemory.Proxy should have comment or be unexported (golint)
  • Line 344: warning: exported method RegistryMemory.PipelineMutator should have comment or be unexported (golint)
  • Line 356: warning: exported method RegistryMemory.WithBrokenPipelineMutator should have comment or be unexported (golint)
  • Line 422: warning: exported method RegistryMemory.Tracer should have comment or be unexported (golint)

• oathkeeper/driver/health/readiness.go
  • Line 4: warning: exported type ReadinessProbe should have comment or be unexported (golint)
  • Line 12: warning: exported type ReadinessProbeEvent should have comment or be unexported (golint)

• oathkeeper/api/decision.go
  • Line 35: warning: exported const DecisionPath should have comment (or a comment on this block) or be unexported (golint)
  • Line 46: warning: exported type DecisionHandler should have comment or be unexported (golint)
  • Line 50: warning: exported function NewJudgeHandler should have comment or be unexported (golint)

• oathkeeper/pipeline/mutate/mutator_id_token.go
  • Line 48: warning: exported type MutatorIDTokenRegistry should have comment or be unexported (golint)
  • Line 52: warning: exported type MutatorIDToken should have comment or be unexported (golint)
  • Line 62: warning: exported type CredentialsIDTokenConfig should have comment or be unexported (golint)
  • Line 69: warning: exported method CredentialsIDTokenConfig.ClaimsTemplateID should have comment or be unexported (golint)
  • Line 73: warning: exported function NewMutatorIDToken should have comment or be unexported (golint)
  • Line 82: warning: exported method MutatorIDToken.GetID should have comment or be unexported (golint)
  • Line 86: warning: exported method MutatorIDToken.WithCache should have comment or be unexported (golint)
  • Line 90: warning: exported method MutatorIDToken.SetCaching should have comment or be unexported (golint)
  • Line 140: warning: exported method MutatorIDToken.Mutate should have comment or be unexported (golint)
  • Line 205: warning: exported method MutatorIDToken.Validate should have comment or be unexported (golint)
  • Line 214: warning: exported method MutatorIDToken.Config should have comment or be unexported (golint)

• oathkeeper/pipeline/authz/keto_engine_acp_ory.go
  • Line 48: warning: exported type AuthorizerKetoEngineACPORYConfiguration should have comment or be unexported (golint)
  • Line 56: warning: exported method AuthorizerKetoEngineACPORYConfiguration.SubjectTemplateID should have comment or be unexported (golint)
  • Line 60: warning: exported method AuthorizerKetoEngineACPORYConfiguration.ActionTemplateID should have comment or be unexported (golint)
  • Line 64: warning: exported method AuthorizerKetoEngineACPORYConfiguration.ResourceTemplateID should have comment or be unexported (golint)
  • Line 68: warning: exported type AuthorizerKetoEngineACPORY should have comment or be unexported (golint)
  • Line 76: warning: exported function NewAuthorizerKetoEngineACPORY should have comment or be unexported (golint)
  • Line 90: warning: exported method AuthorizerKetoEngineACPORY.GetID should have comment or be unexported (golint)
  • Line 96: warning: exported type AuthorizerKetoEngineACPORYRequestBody should have comment or be unexported (golint)
  • Line 103: warning: exported method AuthorizerKetoEngineACPORY.WithContextCreator should have comment or be unexported (golint)
  • Line 107: warning: exported method AuthorizerKetoEngineACPORY.Authorize should have comment or be unexported (golint)
  • Line 208: warning: exported method AuthorizerKetoEngineACPORY.Validate should have comment or be unexported (golint)
  • Line 217: warning: exported method AuthorizerKetoEngineACPORY.Config should have comment or be unexported (golint)

• oathkeeper/x/registry.go
  • Line 9: warning: exported type TestLoggerProvider should have comment or be unexported (golint)
  • Line 11: warning: exported method TestLoggerProvider.Logger should have comment or be unexported (golint)
  • Line 15: warning: exported type RegistryLogger should have comment or be unexported (golint)
  • Line 19: warning: exported type RegistryWriter should have comment or be unexported (golint)

• oathkeeper/pipeline/errors/error_www_authenticate.go
  • Line 16: warning: exported type ErrorWWWAuthenticateConfig should have comment or be unexported (golint)
  • Line 19: warning: exported type ErrorWWWAuthenticate should have comment or be unexported (golint)
  • Line 23: warning: exported type ErrorWWWAuthenticateDependencies should have comment or be unexported (golint)
  • Line 28: warning: exported function NewErrorWWWAuthenticate should have comment or be unexported (golint)
  • Line 35: warning: exported method ErrorWWWAuthenticate.Handle should have comment or be unexported (golint)
  • Line 46: warning: exported method ErrorWWWAuthenticate.Validate should have comment or be unexported (golint)
  • Line 54: warning: exported method ErrorWWWAuthenticate.Config should have comment or be unexported (golint)
  • Line 67: warning: exported method ErrorWWWAuthenticate.GetID should have comment or be unexported (golint)

• oathkeeper/pipeline/errors/registry.go
  • Line 4: warning: exported type Handlers should have comment or be unexported (golint)
  • Line 5: warning: exported type Registry should have comment or be unexported (golint)
  • Line 11: warning: exported method Handlers.IDs should have comment or be unexported (golint)

• oathkeeper/pipeline/rule.go
  • Line 7: warning: exported type Rule should have comment or be unexported (golint)

• oathkeeper/pipeline/authn/authenticator.go
  • Line 17: warning: exported var ErrAuthenticatorNotResponsible should have comment or be unexported (golint)
  • Line 18: warning: exported var ErrAuthenticatorNotEnabled should have comment or be unexported (golint)
  • Line 24: warning: exported type Authenticator should have comment or be unexported (golint)
  • Line 30: warning: exported function NewErrAuthenticatorNotEnabled should have comment or be unexported (golint)
  • Line 34: warning: exported function NewErrAuthenticatorMisconfigured should have comment or be unexported (golint)
  • Line 42: warning: exported type AuthenticationSession should have comment or be unexported (golint)
  • Line 49: warning: exported type MatchContext should have comment or be unexported (golint)
  • Line 56: warning: exported method AuthenticationSession.SetHeader should have comment or be unexported (golint)
  • Line 63: warning: exported method AuthenticationSession.Copy should have comment or be unexported (golint)

• oathkeeper/pipeline/authn/authenticator_bearer_token.go
  • Line 23: warning: exported type AuthenticatorBearerTokenFilter should have comment or be unexported (golint)
  • Line 26: warning: exported type AuthenticatorBearerTokenConfiguration should have comment or be unexported (golint)
  • Line 34: warning: exported type AuthenticatorBearerToken should have comment or be unexported (golint)
  • Line 38: warning: exported function NewAuthenticatorBearerToken should have comment or be unexported (golint)
  • Line 44: warning: exported method AuthenticatorBearerToken.GetID should have comment or be unexported (golint)
  • Line 48: warning: exported method AuthenticatorBearerToken.Validate should have comment or be unexported (golint)
  • Line 57: warning: exported method AuthenticatorBearerToken.Config should have comment or be unexported (golint)
  • Line 74: warning: exported method AuthenticatorBearerToken.Authenticate should have comment or be unexported (golint)

• oathkeeper/pipeline/authn/authenticator_jwt.go
  • Line 20: warning: exported type AuthenticatorJWTRegistry should have comment or be unexported (golint)
  • Line 24: warning: exported type AuthenticatorOAuth2JWTConfiguration should have comment or be unexported (golint)
  • Line 34: warning: exported type AuthenticatorJWT should have comment or be unexported (golint)
  • Line 39: warning: exported function NewAuthenticatorJWT should have comment or be unexported (golint)
  • Line 49: warning: exported method AuthenticatorJWT.GetID should have comment or be unexported (golint)
  • Line 53: warning: exported method AuthenticatorJWT.Validate should have comment or be unexported (golint)
  • Line 62: warning: exported method AuthenticatorJWT.Config should have comment or be unexported (golint)
  • Line 71: warning: exported method AuthenticatorJWT.Authenticate should have comment or be unexported (golint)

• oathkeeper/pipeline/authn/authenticator_unauthorized.go
  • Line 35: warning: exported type AuthenticatorUnauthorized should have comment or be unexported (golint)
  • Line 39: warning: exported function NewAuthenticatorUnauthorized should have comment or be unexported (golint)
  • Line 43: warning: exported method AuthenticatorUnauthorized.Validate should have comment or be unexported (golint)
  • Line 54: warning: exported method AuthenticatorUnauthorized.GetID should have comment or be unexported (golint)
  • Line 58: warning: exported method AuthenticatorUnauthorized.Authenticate should have comment or be unexported (golint)

• oathkeeper/pipeline/authz/authorizer_deny.go
  • Line 36: warning: exported type AuthorizerDeny should have comment or be unexported (golint)
  • Line 40: warning: exported function NewAuthorizerDeny should have comment or be unexported (golint)
  • Line 44: warning: exported method AuthorizerDeny.GetID should have comment or be unexported (golint)
  • Line 48: warning: exported method AuthorizerDeny.Authorize should have comment or be unexported (golint)
  • Line 52: warning: exported method AuthorizerDeny.Validate should have comment or be unexported (golint)

• oathkeeper/pipeline/authn/authenticator_oauth2_client_credentials.go
  • Line 24: warning: exported type AuthenticatorOAuth2Configuration should have comment or be unexported (golint)
  • Line 30: warning: exported type AuthenticatorOAuth2ClientCredentials should have comment or be unexported (golint)
  • Line 35: warning: exported type AuthenticatorOAuth2ClientCredentialsRetryConfiguration should have comment or be unexported (golint)
  • Line 40: warning: exported function NewAuthenticatorOAuth2ClientCredentials should have comment or be unexported (golint)
  • Line 44: warning: exported method AuthenticatorOAuth2ClientCredentials.GetID should have comment or be unexported (golint)
  • Line 48: warning: exported method AuthenticatorOAuth2ClientCredentials.Validate should have comment or be unexported (golint)
  • Line 57: warning: exported method AuthenticatorOAuth2ClientCredentials.Config should have comment or be unexported (golint)
  • Line 92: warning: exported method AuthenticatorOAuth2ClientCredentials.Authenticate should have comment or be unexported (golint)

• oathkeeper/credentials/verifier.go
  • Line 12: warning: exported type Verifier should have comment or be unexported (golint)
  • Line 20: warning: exported type VerifierRegistry should have comment or be unexported (golint)
  • Line 24: warning: exported type ValidationContext should have comment or be unexported (golint)

• oathkeeper/api/credential.go
  • Line 22: warning: exported const CredentialsPath should have comment (or a comment on this block) or be unexported (golint)
  • Line 31: warning: exported type CredentialsHandler should have comment or be unexported (golint)
  • Line 36: warning: exported function NewCredentialHandler should have comment or be unexported (golint)
  • Line 40: warning: exported method CredentialsHandler.SetRoutes should have comment or be unexported (golint)

• oathkeeper/rule/readiness/rule_readiness.go
  • Line 10: warning: exported type RuleReadinessChecker should have comment or be unexported (golint)
  • Line 14: warning: exported type RuleLoadedEvent should have comment or be unexported (golint)
  • Line 17: warning: exported const ProbeName should have comment or be unexported (golint)
  • Line 19: warning: exported var ErrRuleNotYetLoaded should have comment or be unexported (golint)
  • Line 21: warning: exported function NewReadinessHealthChecker should have comment or be unexported (golint)
  • Line 27: warning: exported method RuleReadinessChecker.ID should have comment or be unexported (golint)
  • Line 31: warning: exported method RuleReadinessChecker.Validate should have comment or be unexported (golint)
  • Line 38: warning: exported method RuleReadinessChecker.EventTypes should have comment or be unexported (golint)
  • Line 42: warning: exported method RuleReadinessChecker.EventsReceiver should have comment or be unexported (golint)
  • Line 49: warning: exported method RuleLoadedEvent.ReadinessProbeListenerID should have comment or be unexported (golint)

• oathkeeper/pipeline/mutate/mutator_hydrator.go
  • Line 46: warning: exported const ErrMalformedResponseFromUpstreamAPI should have comment (or a comment on this block) or be unexported (golint)
  • Line 56: warning: exported type MutatorHydrator should have comment or be unexported (golint)
  • Line 65: warning: exported type BasicAuth should have comment or be unexported (golint)
  • Line 92: warning: exported type MutatorHydratorConfig should have comment or be unexported (golint)
  • Line 101: warning: exported function NewMutatorHydrator should have comment or be unexported (golint)
  • Line 113: warning: exported method MutatorHydrator.GetID should have comment or be unexported (golint)
  • Line 144: warning: exported method MutatorHydrator.Mutate should have comment or be unexported (golint)
  • Line 220: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 242: warning: exported method MutatorHydrator.Validate should have comment or be unexported (golint)
  • Line 251: warning: exported method MutatorHydrator.Config should have comment or be unexported (golint)

• oathkeeper/pipeline/mutate/registry.go
  • Line 3: warning: exported type Registry should have comment or be unexported (golint)

• oathkeeper/pipeline/authz/authorizer.go
  • Line 15: warning: exported var ErrAuthorizerNotEnabled should have comment or be unexported (golint)
  • Line 21: warning: exported function NewErrAuthorizerNotEnabled should have comment or be unexported (golint)
  • Line 25: warning: exported function NewErrAuthorizerMisconfigured should have comment or be unexported (golint)
  • Line 33: warning: exported type Authorizer should have comment or be unexported (golint)

• oathkeeper/pipeline/error.go
  • Line 6: warning: exported var ErrPipelineHandlerNotFound should have comment or be unexported (golint)

• oathkeeper/pipeline/mutate/mutator.go
  • Line 35: warning: exported var ErrMutatorNotEnabled should have comment or be unexported (golint)
  • Line 41: warning: exported function NewErrMutatorNotEnabled should have comment or be unexported (golint)
  • Line 45: warning: exported function NewErrMutatorMisconfigured should have comment or be unexported (golint)
  • Line 53: warning: exported type Mutator should have comment or be unexported (golint)

• oathkeeper/pipeline/mutate/mutator_noop.go
  • Line 32: warning: exported type MutatorNoop should have comment or be unexported (golint)
  • Line 34: warning: exported function NewMutatorNoop should have comment or be unexported (golint)
  • Line 38: warning: exported method MutatorNoop.GetID should have comment or be unexported (golint)
  • Line 42: warning: exported method MutatorNoop.Mutate should have comment or be unexported (golint)
  • Line 47: warning: exported method MutatorNoop.Validate should have comment or be unexported (golint)

• oathkeeper/rule/repository.go
  • Line 29: warning: exported type Repository should have comment or be unexported (golint)

• oathkeeper/pipeline/mutate/mutator_broken.go
  • Line 33: warning: exported type MutatorBroken should have comment or be unexported (golint)
  • Line 37: warning: exported function NewMutatorBroken should have comment or be unexported (golint)
  • Line 43: warning: exported method MutatorBroken.GetID should have comment or be unexported (golint)
  • Line 47: warning: exported method MutatorBroken.Mutate should have comment or be unexported (golint)
  • Line 51: warning: exported method MutatorBroken.Validate should have comment or be unexported (golint)

• oathkeeper/pipeline/errors/when.go
  • Line 17: warning: exported type Whens should have comment or be unexported (golint)
  • Line 18: warning: exported type When should have comment or be unexported (golint)
  • Line 23: warning: exported type WhenRequest should have comment or be unexported (golint)
  • Line 28: warning: exported type WhenRequestRemoteIP should have comment or be unexported (golint)
  • Line 33: warning: exported type WhenRequestHeader should have comment or be unexported (golint)
  • Line 43: warning: exported var ErrDoesNotMatchWhen should have comment or be unexported (golint)
  • Line 81: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 203: warning: exported function MatchesWhen should have comment or be unexported (golint)

• oathkeeper/credentials/signer_default.go
  • Line 18: warning: exported type DefaultSigner should have comment or be unexported (golint)
  • Line 22: warning: exported function NewSignerDefault should have comment or be unexported (golint)
  • Line 26: warning: exported method DefaultSigner.Sign should have comment or be unexported (golint)

• oathkeeper/rule/registry.go
  • Line 3: warning: exported type Registry should have comment or be unexported (golint)

• oathkeeper/rule/repository_memory.go
  • Line 46: warning: exported type RepositoryMemory should have comment or be unexported (golint)
  • Line 70: warning: exported function NewRepositoryMemory should have comment or be unexported (golint)
  • Line 85: warning: exported method RepositoryMemory.Count should have comment or be unexported (golint)
  • Line 92: warning: exported method RepositoryMemory.List should have comment or be unexported (golint)
  • Line 100: warning: exported method RepositoryMemory.Get should have comment or be unexported (golint)
  • Line 113: warning: exported method RepositoryMemory.Set should have comment or be unexported (golint)
  • Line 128: warning: exported method RepositoryMemory.Match should have comment or be unexported (golint)

• oathkeeper/driver/configuration/provider.go
  • Line 18: warning: exported const ForbiddenStrategyErrorType should have comment (or a comment on this block) or be unexported (golint)
  • Line 31: warning: exported type Provider should have comment or be unexported (golint)
  • Line 67: warning: exported type ProviderErrorHandlers should have comment or be unexported (golint)
  • Line 72: warning: exported type ProviderAuthenticators should have comment or be unexported (golint)
  • Line 79: warning: exported type ProviderAuthorizers should have comment or be unexported (golint)
  • Line 84: warning: exported type ProviderMutators should have comment or be unexported (golint)

• oathkeeper/helper/bearer.go
  • Line 32: warning: exported type BearerTokenLocation should have comment or be unexported (golint)
  • Line 38: warning: exported function BearerTokenFromRequest should have comment or be unexported (golint)
  • Line 59: warning: exported function DefaultBearerTokenFromRequest should have comment or be unexported (golint)

• oathkeeper/pipeline/authn/authenticator_cookie_session.go
  • Line 27: warning: exported type AuthenticatorCookieSessionFilter should have comment or be unexported (golint)
  • Line 30: warning: exported type AuthenticatorCookieSessionConfiguration should have comment or be unexported (golint)
  • Line 38: warning: exported type AuthenticatorCookieSession should have comment or be unexported (golint)
  • Line 42: warning: exported function NewAuthenticatorCookieSession should have comment or be unexported (golint)
  • Line 48: warning: exported method AuthenticatorCookieSession.GetID should have comment or be unexported (golint)
  • Line 52: warning: exported method AuthenticatorCookieSession.Validate should have comment or be unexported (golint)
  • Line 61: warning: exported method AuthenticatorCookieSession.Config should have comment or be unexported (golint)
  • Line 78: warning: exported method AuthenticatorCookieSession.Authenticate should have comment or be unexported (golint)
  • Line 154: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• oathkeeper/x/version.go
  • Line 3: warning: exported const UnknownVersion should have comment or be unexported (golint)
  • Line 6: warning: exported var Version should have comment or be unexported (golint)

• oathkeeper/credentials/fetcher_default.go
  • Line 1: warning: package comment should be of the form "Package credentials ..." (golint)
  • Line 1: warning: package comment should not have leading space (golint)
  • Line 46: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 57: warning: exported type FetcherDefault should have comment or be unexported (golint)
  • Line 86: warning: exported method FetcherDefault.ResolveSets should have comment or be unexported (golint)
  • Line 134: warning: exported method FetcherDefault.ResolveKey should have comment or be unexported (golint)

• oathkeeper/driver/configuration/provider_viper.go
  • Line 41: warning: exported const ViperKeyProxyReadTimeout should have comment (or a comment on this block) or be unexported (golint)
  • Line 121: warning: exported type ViperProvider should have comment or be unexported (golint)
  • Line 131: warning: exported function NewViperProvider should have comment or be unexported (golint)
  • Line 139: warning: exported method ViperProvider.AccessRuleRepositories should have comment or be unexported (golint)
  • Line 154: warning: exported method ViperProvider.CORSEnabled should have comment or be unexported (golint)
  • Line 158: warning: exported method ViperProvider.CORSOptions should have comment or be unexported (golint)
  • Line 162: warning: exported method ViperProvider.ProxyReadTimeout should have comment or be unexported (golint)
  • Line 166: warning: exported method ViperProvider.ProxyWriteTimeout should have comment or be unexported (golint)
  • Line 170: warning: exported method ViperProvider.ProxyIdleTimeout should have comment or be unexported (golint)
  • Line 174: warning: exported method ViperProvider.ProxyServeAddress should have comment or be unexported (golint)
  • Line 182: warning: exported method ViperProvider.APIReadTimeout should have comment or be unexported (golint)
  • Line 186: warning: exported method ViperProvider.APIWriteTimeout should have comment or be unexported (golint)
  • Line 190: warning: exported method ViperProvider.APIIdleTimeout should have comment or be unexported (golint)
  • Line 194: warning: exported method ViperProvider.APIServeAddress should have comment or be unexported (golint)
  • Line 202: warning: exported method ViperProvider.PrometheusServeAddress should have comment or be unexported (golint)
  • Line 210: warning: exported method ViperProvider.PrometheusMetricsPath should have comment or be unexported (golint)
  • Line 214: warning: exported method ViperProvider.PrometheusCollapseRequestPaths should have comment or be unexported (golint)
  • Line 218: warning: exported method ViperProvider.ParseURLs should have comment or be unexported (golint)
  • Line 241: warning: exported method ViperProvider.ToScopeStrategy should have comment or be unexported (golint)
  • Line 298: warning: exported method ViperProvider.PipelineConfig should have comment or be unexported (golint)
  • Line 379: warning: exported method ViperProvider.ErrorHandlerConfig should have comment or be unexported (golint)
  • Line 383: warning: exported method ViperProvider.ErrorHandlerFallbackSpecificity should have comment or be unexported (golint)
  • Line 387: warning: exported method ViperProvider.ErrorHandlerIsEnabled should have comment or be unexported (golint)
  • Line 391: warning: exported method ViperProvider.AuthenticatorIsEnabled should have comment or be unexported (golint)
  • Line 395: warning: exported method ViperProvider.AuthenticatorConfig should have comment or be unexported (golint)
  • Line 399: warning: exported method ViperProvider.AuthenticatorJwtJwkMaxWait should have comment or be unexported (golint)
  • Line 403: warning: exported method ViperProvider.AuthenticatorJwtJwkTtl should have comment or be unexported (golint)
  • Line 407: warning: exported method ViperProvider.AuthorizerIsEnabled should have comment or be unexported (golint)
  • Line 411: warning: exported method ViperProvider.AuthorizerConfig should have comment or be unexported (golint)
  • Line 415: warning: exported method ViperProvider.MutatorIsEnabled should have comment or be unexported (golint)
  • Line 419: warning: exported method ViperProvider.MutatorConfig should have comment or be unexported (golint)
  • Line 423: warning: exported method ViperProvider.JSONWebKeyURLs should have comment or be unexported (golint)
  • Line 427: warning: exported method ViperProvider.TracingServiceName should have comment or be unexported (golint)
  • Line 431: warning: exported method ViperProvider.TracingProvider should have comment or be unexported (golint)
  • Line 435: warning: exported method ViperProvider.TracingJaegerConfig should have comment or be unexported (golint)

• oathkeeper/cmd/root.go
  • Line 32: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)

• oathkeeper/rule/validator.go
  • Line 41: warning: exported type Validator should have comment or be unexported (golint)
  • Line 47: warning: exported type ValidatorDefault should have comment or be unexported (golint)
  • Line 51: warning: exported function NewValidatorDefault should have comment or be unexported (golint)
  • Line 123: warning: exported method ValidatorDefault.Validate 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!

misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!