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!

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.

• hydra/test/conformance/run_test.go
  • Line 195: warning: cyclomatic complexity 18 of function createPlan() is high (> 15) (gocyclo)

• hydra/x/oauth2cors/cors.go
  • Line 41: warning: cyclomatic complexity 19 of function Middleware() is high (> 15) (gocyclo)

• hydra/oauth2/oauth2_refresh_token_test.go
  • Line 29: warning: cyclomatic complexity 18 of function TestCreateRefreshTokenSessionStress() is high (> 15) (gocyclo)

• hydra/consent/manager_test_helpers.go
  • Line 261: warning: cyclomatic complexity 30 of function ManagerTests() is high (> 15) (gocyclo)

• hydra/consent/strategy_default.go
  • Line 347: warning: cyclomatic complexity 27 of function (*DefaultStrategy).verifyAuthentication() is high (> 15) (gocyclo)
  • Line 739: warning: cyclomatic complexity 26 of function (*DefaultStrategy).issueLogoutVerifier() is high (> 15) (gocyclo)

• hydra/client/validator.go
  • Line 73: warning: cyclomatic complexity 37 of function (*Validator).Validate() is high (> 15) (gocyclo)

• hydra/persistence/sql/migratest/migration_test.go
  • Line 29: warning: cyclomatic complexity 21 of function TestMigrations() is high (> 15) (gocyclo)

• hydra/test/mock-client/main.go
  • Line 157: warning: cyclomatic complexity 19 of function checkTokenResponse() is high (> 15) (gocyclo)

golint35%

Golint is a linter for Go source code.

• hydra/internal/fizzmigrate/consent/sql_migration_files.go
  • Line 933: warning: don't use underscores in Go names; var _consentMigrationsSqlTests10_testSql should be _consentMigrationsSQLTests10TestSQL (golint)
  • Line 935: warning: don't use underscores in Go names; func consentMigrationsSqlTests10_testSqlBytes should be consentMigrationsSQLTests10TestSQLBytes (golint)
  • Line 942: warning: don't use underscores in Go names; func consentMigrationsSqlTests10_testSql should be consentMigrationsSQLTests10TestSQL (golint)
  • Line 953: warning: don't use underscores in Go names; var _consentMigrationsSqlTests11_testSql should be _consentMigrationsSQLTests11TestSQL (golint)
  • Line 955: warning: don't use underscores in Go names; func consentMigrationsSqlTests11_testSqlBytes should be consentMigrationsSQLTests11TestSQLBytes (golint)
  • Line 962: warning: don't use underscores in Go names; func consentMigrationsSqlTests11_testSql should be consentMigrationsSQLTests11TestSQL (golint)
  • Line 973: warning: don't use underscores in Go names; var _consentMigrationsSqlTests12_testSql should be _consentMigrationsSQLTests12TestSQL (golint)
  • Line 975: warning: don't use underscores in Go names; func consentMigrationsSqlTests12_testSqlBytes should be consentMigrationsSQLTests12TestSQLBytes (golint)
  • Line 982: warning: don't use underscores in Go names; func consentMigrationsSqlTests12_testSql should be consentMigrationsSQLTests12TestSQL (golint)
  • Line 993: warning: don't use underscores in Go names; var _consentMigrationsSqlTests13_testSql should be _consentMigrationsSQLTests13TestSQL (golint)
  • Line 995: warning: don't use underscores in Go names; func consentMigrationsSqlTests13_testSqlBytes should be consentMigrationsSQLTests13TestSQLBytes (golint)
  • Line 1002: warning: don't use underscores in Go names; func consentMigrationsSqlTests13_testSql should be consentMigrationsSQLTests13TestSQL (golint)
  • Line 1013: warning: don't use underscores in Go names; var _consentMigrationsSqlTests14_testSql should be _consentMigrationsSQLTests14TestSQL (golint)
  • Line 1015: warning: don't use underscores in Go names; func consentMigrationsSqlTests14_testSqlBytes should be consentMigrationsSQLTests14TestSQLBytes (golint)
  • Line 1022: warning: don't use underscores in Go names; func consentMigrationsSqlTests14_testSql should be consentMigrationsSQLTests14TestSQL (golint)
  • Line 1033: warning: don't use underscores in Go names; var _consentMigrationsSqlTests1_testSql should be _consentMigrationsSQLTests1TestSQL (golint)
  • Line 1035: warning: don't use underscores in Go names; func consentMigrationsSqlTests1_testSqlBytes should be consentMigrationsSQLTests1TestSQLBytes (golint)
  • Line 1042: warning: don't use underscores in Go names; func consentMigrationsSqlTests1_testSql should be consentMigrationsSQLTests1TestSQL (golint)
  • Line 1053: warning: don't use underscores in Go names; var _consentMigrationsSqlTests2_testSql should be _consentMigrationsSQLTests2TestSQL (golint)
  • Line 1055: warning: don't use underscores in Go names; func consentMigrationsSqlTests2_testSqlBytes should be consentMigrationsSQLTests2TestSQLBytes (golint)
  • Line 1062: warning: don't use underscores in Go names; func consentMigrationsSqlTests2_testSql should be consentMigrationsSQLTests2TestSQL (golint)
  • Line 1073: warning: don't use underscores in Go names; var _consentMigrationsSqlTests3_testSql should be _consentMigrationsSQLTests3TestSQL (golint)
  • Line 1075: warning: don't use underscores in Go names; func consentMigrationsSqlTests3_testSqlBytes should be consentMigrationsSQLTests3TestSQLBytes (golint)
  • Line 1082: warning: don't use underscores in Go names; func consentMigrationsSqlTests3_testSql should be consentMigrationsSQLTests3TestSQL (golint)
  • Line 1093: warning: don't use underscores in Go names; var _consentMigrationsSqlTests4_testSql should be _consentMigrationsSQLTests4TestSQL (golint)
  • Line 1095: warning: don't use underscores in Go names; func consentMigrationsSqlTests4_testSqlBytes should be consentMigrationsSQLTests4TestSQLBytes (golint)
  • Line 1102: warning: don't use underscores in Go names; func consentMigrationsSqlTests4_testSql should be consentMigrationsSQLTests4TestSQL (golint)
  • Line 1113: warning: don't use underscores in Go names; var _consentMigrationsSqlTests5_testSql should be _consentMigrationsSQLTests5TestSQL (golint)
  • Line 1115: warning: don't use underscores in Go names; func consentMigrationsSqlTests5_testSqlBytes should be consentMigrationsSQLTests5TestSQLBytes (golint)
  • Line 1122: warning: don't use underscores in Go names; func consentMigrationsSqlTests5_testSql should be consentMigrationsSQLTests5TestSQL (golint)
  • Line 1133: warning: don't use underscores in Go names; var _consentMigrationsSqlTests6_testSql should be _consentMigrationsSQLTests6TestSQL (golint)
  • Line 1135: warning: don't use underscores in Go names; func consentMigrationsSqlTests6_testSqlBytes should be consentMigrationsSQLTests6TestSQLBytes (golint)
  • Line 1142: warning: don't use underscores in Go names; func consentMigrationsSqlTests6_testSql should be consentMigrationsSQLTests6TestSQL (golint)
  • Line 1153: warning: don't use underscores in Go names; var _consentMigrationsSqlTests7_testSql should be _consentMigrationsSQLTests7TestSQL (golint)
  • Line 1155: warning: don't use underscores in Go names; func consentMigrationsSqlTests7_testSqlBytes should be consentMigrationsSQLTests7TestSQLBytes (golint)
  • Line 1162: warning: don't use underscores in Go names; func consentMigrationsSqlTests7_testSql should be consentMigrationsSQLTests7TestSQL (golint)
  • Line 1173: warning: don't use underscores in Go names; var _consentMigrationsSqlTests8_testSql should be _consentMigrationsSQLTests8TestSQL (golint)
  • Line 1175: warning: don't use underscores in Go names; func consentMigrationsSqlTests8_testSqlBytes should be consentMigrationsSQLTests8TestSQLBytes (golint)
  • Line 1182: warning: don't use underscores in Go names; func consentMigrationsSqlTests8_testSql should be consentMigrationsSQLTests8TestSQL (golint)
  • Line 1193: warning: don't use underscores in Go names; var _consentMigrationsSqlTests9_testSql should be _consentMigrationsSQLTests9TestSQL (golint)
  • Line 1195: warning: don't use underscores in Go names; func consentMigrationsSqlTests9_testSqlBytes should be consentMigrationsSQLTests9TestSQLBytes (golint)
  • Line 1202: warning: don't use underscores in Go names; func consentMigrationsSqlTests9_testSql should be consentMigrationsSQLTests9TestSQL (golint)

• hydra/cmd/clients_list.go
  • Line 9: warning: exported function NewClientsListCmd should have comment or be unexported (golint)

• hydra/cmd/keys.go
  • Line 30: warning: comment on exported function NewKeysCmd should be of the form "NewKeysCmd ..." (golint)

• hydra/x/redirect_uri.go
  • Line 13: warning: exported function IsRedirectURISecure should have comment or be unexported (golint)

• hydra/driver/config/helper.go
  • Line 9: warning: exported function MustValidate should have comment or be unexported (golint)

• hydra/x/tls_termination.go
  • Line 18: warning: exported function MatchesRange should have comment or be unexported (golint)
  • Line 55: warning: exported function RejectInsecureRequests should have comment or be unexported (golint)

• hydra/driver/registry_base.go
  • Line 43: warning: exported type RegistryBase should have comment or be unexported (golint)
  • Line 84: warning: exported method RegistryBase.WithBuildInfo should have comment or be unexported (golint)
  • Line 91: warning: exported method RegistryBase.OAuth2AwareMiddleware should have comment or be unexported (golint)
  • Line 98: warning: exported method RegistryBase.RegisterRoutes should have comment or be unexported (golint)
  • Line 112: warning: exported method RegistryBase.BuildVersion should have comment or be unexported (golint)
  • Line 116: warning: exported method RegistryBase.BuildDate should have comment or be unexported (golint)
  • Line 120: warning: exported method RegistryBase.BuildHash should have comment or be unexported (golint)
  • Line 124: warning: exported method RegistryBase.WithConfig should have comment or be unexported (golint)
  • Line 129: warning: exported method RegistryBase.Writer should have comment or be unexported (golint)
  • Line 138: warning: exported method RegistryBase.WithLogger should have comment or be unexported (golint)
  • Line 143: warning: exported method RegistryBase.Logger should have comment or be unexported (golint)
  • Line 150: warning: exported method RegistryBase.AuditLogger should have comment or be unexported (golint)
  • Line 157: warning: exported method RegistryBase.ClientHasher should have comment or be unexported (golint)
  • Line 168: warning: exported method RegistryBase.ClientHandler should have comment or be unexported (golint)
  • Line 175: warning: exported method RegistryBase.ClientValidator should have comment or be unexported (golint)
  • Line 182: warning: exported method RegistryBase.KeyHandler should have comment or be unexported (golint)
  • Line 189: warning: exported method RegistryBase.HealthHandler should have comment or be unexported (golint)
  • Line 215: warning: exported method RegistryBase.ConsentStrategy should have comment or be unexported (golint)
  • Line 222: warning: exported method RegistryBase.KeyGenerators should have comment or be unexported (golint)
  • Line 234: warning: exported method RegistryBase.KeyCipher should have comment or be unexported (golint)
  • Line 241: warning: exported method RegistryBase.CookieStore should have comment or be unexported (golint)
  • Line 275: warning: exported method RegistryBase.OAuth2HMACStrategy should have comment or be unexported (golint)
  • Line 282: warning: exported method RegistryBase.OAuth2Provider should have comment or be unexported (golint)
  • Line 331: warning: exported method RegistryBase.ScopeStrategy should have comment or be unexported (golint)
  • Line 369: warning: exported method RegistryBase.AccessTokenJWTStrategy should have comment or be unexported (golint)
  • Line 376: warning: exported method RegistryBase.OpenIDJWTStrategy should have comment or be unexported (golint)
  • Line 383: warning: exported method RegistryBase.FositeOpenIDDefaultStrategy should have comment or be unexported (golint)
  • Line 392: warning: exported method RegistryBase.OpenIDConnectRequestValidator should have comment or be unexported (golint)
  • Line 399: warning: exported method RegistryBase.AudienceStrategy should have comment or be unexported (golint)
  • Line 403: warning: exported method RegistryBase.ConsentHandler should have comment or be unexported (golint)
  • Line 410: warning: exported method RegistryBase.OAuth2Handler should have comment or be unexported (golint)
  • Line 417: warning: exported method RegistryBase.SubjectIdentifierAlgorithm should have comment or be unexported (golint)
  • Line 432: warning: exported method RegistryBase.Tracer should have comment or be unexported (golint)
  • Line 444: warning: exported method RegistryBase.PrometheusManager should have comment or be unexported (golint)
  • Line 451: warning: exported method RegistryBase.Persister should have comment or be unexported (golint)
  • Line 455: warning: exported method RegistryBase.Config should have comment or be unexported (golint)

• hydra/x/clean_sql.go
  • Line 11: warning: exported function CleanSQL should have comment or be unexported (golint)
  • Line 43: warning: exported function CleanSQLPop should have comment or be unexported (golint)

• hydra/jwk/aead.go
  • Line 34: warning: exported type AEAD should have comment or be unexported (golint)
  • Line 38: warning: exported function NewAEAD should have comment or be unexported (golint)
  • Line 48: warning: exported method AEAD.Encrypt should have comment or be unexported (golint)
  • Line 66: warning: exported method AEAD.Decrypt should have comment or be unexported (golint)

• hydra/internal/fizzmigrate/client/sql_migration_files.go
  • Line 870: warning: don't use underscores in Go names; var _clientMigrationsSqlTests10_testSql should be _clientMigrationsSQLTests10TestSQL (golint)
  • Line 872: warning: don't use underscores in Go names; func clientMigrationsSqlTests10_testSqlBytes should be clientMigrationsSQLTests10TestSQLBytes (golint)
  • Line 879: warning: don't use underscores in Go names; func clientMigrationsSqlTests10_testSql should be clientMigrationsSQLTests10TestSQL (golint)
  • Line 890: warning: don't use underscores in Go names; var _clientMigrationsSqlTests11_testSql should be _clientMigrationsSQLTests11TestSQL (golint)
  • Line 892: warning: don't use underscores in Go names; func clientMigrationsSqlTests11_testSqlBytes should be clientMigrationsSQLTests11TestSQLBytes (golint)
  • Line 899: warning: don't use underscores in Go names; func clientMigrationsSqlTests11_testSql should be clientMigrationsSQLTests11TestSQL (golint)
  • Line 910: warning: don't use underscores in Go names; var _clientMigrationsSqlTests12_testSql should be _clientMigrationsSQLTests12TestSQL (golint)
  • Line 912: warning: don't use underscores in Go names; func clientMigrationsSqlTests12_testSqlBytes should be clientMigrationsSQLTests12TestSQLBytes (golint)
  • Line 919: warning: don't use underscores in Go names; func clientMigrationsSqlTests12_testSql should be clientMigrationsSQLTests12TestSQL (golint)
  • Line 930: warning: don't use underscores in Go names; var _clientMigrationsSqlTests13_testSql should be _clientMigrationsSQLTests13TestSQL (golint)
  • Line 932: warning: don't use underscores in Go names; func clientMigrationsSqlTests13_testSqlBytes should be clientMigrationsSQLTests13TestSQLBytes (golint)
  • Line 939: warning: don't use underscores in Go names; func clientMigrationsSqlTests13_testSql should be clientMigrationsSQLTests13TestSQL (golint)
  • Line 950: warning: don't use underscores in Go names; var _clientMigrationsSqlTests14_testSql should be _clientMigrationsSQLTests14TestSQL (golint)
  • Line 952: warning: don't use underscores in Go names; func clientMigrationsSqlTests14_testSqlBytes should be clientMigrationsSQLTests14TestSQLBytes (golint)
  • Line 959: warning: don't use underscores in Go names; func clientMigrationsSqlTests14_testSql should be clientMigrationsSQLTests14TestSQL (golint)
  • Line 970: warning: don't use underscores in Go names; var _clientMigrationsSqlTests1_testSql should be _clientMigrationsSQLTests1TestSQL (golint)
  • Line 972: warning: don't use underscores in Go names; func clientMigrationsSqlTests1_testSqlBytes should be clientMigrationsSQLTests1TestSQLBytes (golint)
  • Line 979: warning: don't use underscores in Go names; func clientMigrationsSqlTests1_testSql should be clientMigrationsSQLTests1TestSQL (golint)
  • Line 990: warning: don't use underscores in Go names; var _clientMigrationsSqlTests2_testSql should be _clientMigrationsSQLTests2TestSQL (golint)
  • Line 992: warning: don't use underscores in Go names; func clientMigrationsSqlTests2_testSqlBytes should be clientMigrationsSQLTests2TestSQLBytes (golint)
  • Line 999: warning: don't use underscores in Go names; func clientMigrationsSqlTests2_testSql should be clientMigrationsSQLTests2TestSQL (golint)
  • Line 1010: warning: don't use underscores in Go names; var _clientMigrationsSqlTests3_testSql should be _clientMigrationsSQLTests3TestSQL (golint)
  • Line 1012: warning: don't use underscores in Go names; func clientMigrationsSqlTests3_testSqlBytes should be clientMigrationsSQLTests3TestSQLBytes (golint)
  • Line 1019: warning: don't use underscores in Go names; func clientMigrationsSqlTests3_testSql should be clientMigrationsSQLTests3TestSQL (golint)
  • Line 1030: warning: don't use underscores in Go names; var _clientMigrationsSqlTests4_testSql should be _clientMigrationsSQLTests4TestSQL (golint)
  • Line 1032: warning: don't use underscores in Go names; func clientMigrationsSqlTests4_testSqlBytes should be clientMigrationsSQLTests4TestSQLBytes (golint)
  • Line 1039: warning: don't use underscores in Go names; func clientMigrationsSqlTests4_testSql should be clientMigrationsSQLTests4TestSQL (golint)
  • Line 1050: warning: don't use underscores in Go names; var _clientMigrationsSqlTests5_testSql should be _clientMigrationsSQLTests5TestSQL (golint)
  • Line 1052: warning: don't use underscores in Go names; func clientMigrationsSqlTests5_testSqlBytes should be clientMigrationsSQLTests5TestSQLBytes (golint)
  • Line 1059: warning: don't use underscores in Go names; func clientMigrationsSqlTests5_testSql should be clientMigrationsSQLTests5TestSQL (golint)
  • Line 1070: warning: don't use underscores in Go names; var _clientMigrationsSqlTests6_testSql should be _clientMigrationsSQLTests6TestSQL (golint)
  • Line 1072: warning: don't use underscores in Go names; func clientMigrationsSqlTests6_testSqlBytes should be clientMigrationsSQLTests6TestSQLBytes (golint)
  • Line 1079: warning: don't use underscores in Go names; func clientMigrationsSqlTests6_testSql should be clientMigrationsSQLTests6TestSQL (golint)
  • Line 1090: warning: don't use underscores in Go names; var _clientMigrationsSqlTests7_testSql should be _clientMigrationsSQLTests7TestSQL (golint)
  • Line 1092: warning: don't use underscores in Go names; func clientMigrationsSqlTests7_testSqlBytes should be clientMigrationsSQLTests7TestSQLBytes (golint)
  • Line 1099: warning: don't use underscores in Go names; func clientMigrationsSqlTests7_testSql should be clientMigrationsSQLTests7TestSQL (golint)
  • Line 1110: warning: don't use underscores in Go names; var _clientMigrationsSqlTests8_testSql should be _clientMigrationsSQLTests8TestSQL (golint)
  • Line 1112: warning: don't use underscores in Go names; func clientMigrationsSqlTests8_testSqlBytes should be clientMigrationsSQLTests8TestSQLBytes (golint)
  • Line 1119: warning: don't use underscores in Go names; func clientMigrationsSqlTests8_testSql should be clientMigrationsSQLTests8TestSQL (golint)
  • Line 1130: warning: don't use underscores in Go names; var _clientMigrationsSqlTests9_testSql should be _clientMigrationsSQLTests9TestSQL (golint)
  • Line 1132: warning: don't use underscores in Go names; func clientMigrationsSqlTests9_testSqlBytes should be clientMigrationsSQLTests9TestSQLBytes (golint)
  • Line 1139: warning: don't use underscores in Go names; func clientMigrationsSqlTests9_testSql should be clientMigrationsSQLTests9TestSQL (golint)

• hydra/cmd/cli/handler_jwk.go
  • Line 44: warning: exported type JWKHandler should have comment or be unexported (golint)
  • Line 50: warning: exported method JWKHandler.CreateKeys should have comment or be unexported (golint)
  • Line 92: warning: exported method JWKHandler.ImportKeys should have comment or be unexported (golint)
  • Line 168: warning: exported method JWKHandler.GetKeys should have comment or be unexported (golint)
  • Line 177: warning: exported method JWKHandler.DeleteKeys should have comment or be unexported (golint)

• hydra/persistence/sql/persister_oauth2.go
  • Line 32: warning: exported type OAuth2RequestSQL should have comment or be unexported (golint)
  • Line 58: warning: exported method OAuth2RequestSQL.TableName should have comment or be unexported (golint)
  • Line 160: warning: exported method Persister.ClientAssertionJWTValid should have comment or be unexported (golint)
  • Line 176: warning: exported method Persister.SetClientAssertionJWT should have comment or be unexported (golint)
  • Line 199: warning: exported method Persister.GetClientAssertionJWT should have comment or be unexported (golint)
  • Line 204: warning: exported method Persister.SetClientAssertionJWTRaw should have comment or be unexported (golint)
  • Line 290: warning: exported method Persister.CreateAuthorizeCodeSession should have comment or be unexported (golint)
  • Line 294: warning: exported method Persister.GetAuthorizeCodeSession should have comment or be unexported (golint)
  • Line 298: warning: exported method Persister.InvalidateAuthorizeCodeSession should have comment or be unexported (golint)
  • Line 308: warning: exported method Persister.CreateAccessTokenSession should have comment or be unexported (golint)
  • Line 312: warning: exported method Persister.GetAccessTokenSession should have comment or be unexported (golint)
  • Line 316: warning: exported method Persister.DeleteAccessTokenSession should have comment or be unexported (golint)
  • Line 320: warning: exported method Persister.CreateRefreshTokenSession should have comment or be unexported (golint)
  • Line 324: warning: exported method Persister.GetRefreshTokenSession should have comment or be unexported (golint)
  • Line 328: warning: exported method Persister.DeleteRefreshTokenSession should have comment or be unexported (golint)
  • Line 332: warning: exported method Persister.CreateOpenIDConnectSession should have comment or be unexported (golint)
  • Line 336: warning: exported method Persister.GetOpenIDConnectSession should have comment or be unexported (golint)
  • Line 340: warning: exported method Persister.DeleteOpenIDConnectSession should have comment or be unexported (golint)
  • Line 344: warning: exported method Persister.GetPKCERequestSession should have comment or be unexported (golint)
  • Line 348: warning: exported method Persister.CreatePKCERequestSession should have comment or be unexported (golint)
  • Line 352: warning: exported method Persister.DeletePKCERequestSession should have comment or be unexported (golint)
  • Line 356: warning: exported method Persister.RevokeRefreshToken should have comment or be unexported (golint)
  • Line 360: warning: exported method Persister.RevokeAccessToken should have comment or be unexported (golint)
  • Line 364: warning: exported method Persister.FlushInactiveAccessTokens should have comment or be unexported (golint)
  • Line 377: warning: exported method Persister.FlushInactiveRefreshTokens should have comment or be unexported (golint)
  • Line 391: warning: exported method Persister.DeleteAccessTokens should have comment or be unexported (golint)

• hydra/x/const.go
  • Line 4: warning: exported const OpenIDConnectKeyName should have comment (or a comment on this block) or be unexported (golint)

• hydra/client/error.go
  • Line 9: warning: exported var ErrInvalidClientMetadata should have comment or be unexported (golint)
  • Line 15: warning: exported var ErrInvalidRedirectURI should have comment or be unexported (golint)

• hydra/cmd/clients_delete.go
  • Line 29: warning: comment on exported function NewClientsDeleteCmd should be of the form "NewClientsDeleteCmd ..." (golint)

• hydra/cmd/serve_public.go
  • Line 23: warning: comment on exported function NewServePublicCmd should be of the form "NewServePublicCmd ..." (golint)

• hydra/x/basic_auth.go
  • Line 28: warning: exported function BasicAuth should have comment or be unexported (golint)

• hydra/consent/handler.go
  • Line 42: warning: exported type Handler should have comment or be unexported (golint)
  • Line 48: warning: exported const LoginPath should have comment (or a comment on this block) or be unexported (golint)
  • Line 54: warning: exported function NewHandler should have comment or be unexported (golint)
  • Line 64: warning: exported method Handler.SetRoutes should have comment or be unexported (golint)
  • Line 82: warning: comment on exported method Handler.DeleteConsentSession should be of the form "DeleteConsentSession ..." (golint)
  • Line 130: warning: comment on exported method Handler.GetConsentSessions should be of the form "GetConsentSessions ..." (golint)
  • Line 192: warning: comment on exported method Handler.DeleteLoginSession should be of the form "DeleteLoginSession ..." (golint)
  • Line 229: warning: comment on exported method Handler.GetLoginRequest should be of the form "GetLoginRequest ..." (golint)
  • Line 283: warning: comment on exported method Handler.AcceptLoginRequest should be of the form "AcceptLoginRequest ..." (golint)
  • Line 376: warning: comment on exported method Handler.RejectLoginRequest should be of the form "RejectLoginRequest ..." (golint)
  • Line 454: warning: comment on exported method Handler.GetConsentRequest should be of the form "GetConsentRequest ..." (golint)
  • Line 516: warning: comment on exported method Handler.AcceptConsentRequest should be of the form "AcceptConsentRequest ..." (golint)
  • Line 596: warning: comment on exported method Handler.RejectConsentRequest should be of the form "RejectConsentRequest ..." (golint)
  • Line 676: warning: comment on exported method Handler.AcceptLogoutRequest should be of the form "AcceptLogoutRequest ..." (golint)
  • Line 711: warning: comment on exported method Handler.RejectLogoutRequest should be of the form "RejectLogoutRequest ..." (golint)
  • Line 743: warning: comment on exported method Handler.GetLogoutRequest should be of the form "GetLogoutRequest ..." (golint)

• hydra/x/error_enhancer.go
  • Line 17: warning: exported function ErrorEnhancer should have comment or be unexported (golint)

• hydra/client/client.go
  • Line 203: warning: exported method Client.TableName should have comment or be unexported (golint)
  • Line 207: warning: exported method Client.BeforeSave should have comment or be unexported (golint)
  • Line 237: warning: exported method Client.GetID should have comment or be unexported (golint)
  • Line 241: warning: exported method Client.GetRedirectURIs should have comment or be unexported (golint)
  • Line 245: warning: exported method Client.GetHashedSecret should have comment or be unexported (golint)
  • Line 249: warning: exported method Client.GetScopes should have comment or be unexported (golint)
  • Line 253: warning: exported method Client.GetAudience should have comment or be unexported (golint)
  • Line 257: warning: exported method Client.GetGrantTypes should have comment or be unexported (golint)
  • Line 269: warning: exported method Client.GetResponseTypes should have comment or be unexported (golint)
  • Line 281: warning: exported method Client.GetResponseModes should have comment or be unexported (golint)
  • Line 290: warning: exported method Client.GetOwner should have comment or be unexported (golint)
  • Line 294: warning: exported method Client.IsPublic should have comment or be unexported (golint)
  • Line 298: warning: exported method Client.GetJSONWebKeysURI should have comment or be unexported (golint)
  • Line 302: warning: exported method Client.GetJSONWebKeys should have comment or be unexported (golint)
  • Line 309: warning: exported method Client.GetTokenEndpointAuthSigningAlgorithm should have comment or be unexported (golint)
  • Line 316: warning: exported method Client.GetRequestObjectSigningAlgorithm should have comment or be unexported (golint)
  • Line 320: warning: exported method Client.GetTokenEndpointAuthMethod should have comment or be unexported (golint)
  • Line 327: warning: exported method Client.GetRequestURIs should have comment or be unexported (golint)

• hydra/jwk/generator_ecdsa521.go
  • Line 34: warning: exported type ECDSA512Generator should have comment or be unexported (golint)
  • Line 36: warning: exported method ECDSA512Generator.Generate should have comment or be unexported (golint)

• hydra/x/test_helpers.go
  • Line 27: warning: exported function FositeStore should have comment or be unexported (golint)

• hydra/jwk/generator_ecdsa256.go
  • Line 34: warning: exported type ECDSA256Generator should have comment or be unexported (golint)
  • Line 36: warning: exported method ECDSA256Generator.Generate should have comment or be unexported (golint)

• hydra/persistence/sql/persister_consent.go
  • Line 25: warning: exported method Persister.RevokeSubjectConsentSession should have comment or be unexported (golint)
  • Line 29: warning: exported method Persister.RevokeSubjectClientConsentSession should have comment or be unexported (golint)
  • Line 86: warning: exported method Persister.RevokeSubjectLoginSession should have comment or be unexported (golint)
  • Line 103: warning: exported method Persister.CreateForcedObfuscatedLoginSession should have comment or be unexported (golint)
  • Line 122: warning: exported method Persister.GetForcedObfuscatedLoginSession should have comment or be unexported (golint)
  • Line 138: warning: exported method Persister.CreateConsentRequest should have comment or be unexported (golint)
  • Line 142: warning: exported method Persister.GetConsentRequest should have comment or be unexported (golint)
  • Line 155: warning: exported method Persister.CreateLoginRequest should have comment or be unexported (golint)
  • Line 159: warning: exported method Persister.GetLoginRequest should have comment or be unexported (golint)
  • Line 173: warning: exported method Persister.HandleConsentRequest should have comment or be unexported (golint)
  • Line 196: warning: exported method Persister.VerifyAndInvalidateConsentRequest should have comment or be unexported (golint)
  • Line 217: warning: exported method Persister.HandleLoginRequest should have comment or be unexported (golint)
  • Line 229: warning: exported method Persister.VerifyAndInvalidateLoginRequest should have comment or be unexported (golint)
  • Line 250: warning: exported method Persister.GetRememberedLoginSession should have comment or be unexported (golint)
  • Line 262: warning: exported method Persister.ConfirmLoginSession should have comment or be unexported (golint)
  • Line 272: warning: exported method Persister.CreateLoginSession should have comment or be unexported (golint)
  • Line 276: warning: exported method Persister.DeleteLoginSession should have comment or be unexported (golint)
  • Line 283: warning: exported method Persister.FindGrantedAndRememberedConsentRequests should have comment or be unexported (golint)
  • Line 307: warning: exported method Persister.FindSubjectsGrantedConsentRequests should have comment or be unexported (golint)
  • Line 327: warning: exported method Persister.CountSubjectsGrantedConsentRequests should have comment or be unexported (golint)
  • Line 366: warning: exported method Persister.ListUserAuthenticatedClientsWithFrontChannelLogout should have comment or be unexported (golint)
  • Line 370: warning: exported method Persister.ListUserAuthenticatedClientsWithBackChannelLogout should have comment or be unexported (golint)
  • Line 393: warning: exported method Persister.CreateLogoutRequest should have comment or be unexported (golint)
  • Line 397: warning: exported method Persister.AcceptLogoutRequest should have comment or be unexported (golint)
  • Line 405: warning: exported method Persister.RejectLogoutRequest should have comment or be unexported (golint)
  • Line 412: warning: exported method Persister.GetLogoutRequest should have comment or be unexported (golint)
  • Line 417: warning: exported method Persister.VerifyAndInvalidateLogoutRequest should have comment or be unexported (golint)
  • Line 442: warning: exported method Persister.FlushInactiveLoginConsentRequests should have comment or be unexported (golint)

• hydra/x/router.go
  • Line 9: warning: exported type RouterAdmin should have comment or be unexported (golint)
  • Line 13: warning: exported type RouterPublic should have comment or be unexported (golint)
  • Line 17: warning: exported method RouterPublic.RouterAdmin should have comment or be unexported (golint)
  • Line 21: warning: exported method RouterAdmin.RouterPublic should have comment or be unexported (golint)
  • Line 25: warning: exported function NewRouterPublic should have comment or be unexported (golint)
  • Line 33: warning: exported function NewRouterAdmin should have comment or be unexported (golint)

• hydra/jwk/generator.go
  • Line 25: warning: exported type KeyGenerator should have comment or be unexported (golint)

• hydra/oauth2/fosite_store_helpers.go
  • Line 55: warning: exported type BlacklistedJTI should have comment or be unexported (golint)
  • Line 61: warning: exported method BlacklistedJTI.AfterFind should have comment or be unexported (golint)
  • Line 66: warning: exported method BlacklistedJTI.TableName should have comment or be unexported (golint)
  • Line 70: warning: exported function NewBlacklistedJTI should have comment or be unexported (golint)
  • Line 79: warning: exported type AssertionJWTReader should have comment or be unexported (golint)

• hydra/cmd/cli/handler.go
  • Line 34: warning: exported type Handler should have comment or be unexported (golint)
  • Line 43: warning: exported function Remote should have comment or be unexported (golint)
  • Line 54: warning: exported function RemoteURI should have comment or be unexported (golint)
  • Line 60: warning: exported function NewHandler should have comment or be unexported (golint)

• hydra/consent/types.go
  • Line 46: warning: comment on exported type RequestHandlerResponse should be of the form "RequestHandlerResponse ..." (with optional leading article) (golint)
  • Line 56: warning: comment on exported type LoginSession should be of the form "LoginSession ..." (with optional leading article) (golint)
  • Line 64: warning: exported method LoginSession.TableName should have comment or be unexported (golint)
  • Line 64: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 68: warning: comment on exported type RequestDeniedError should be of the form "RequestDeniedError ..." (with optional leading article) (golint)
  • Line 95: warning: exported method RequestDeniedError.IsError should have comment or be unexported (golint)
  • Line 99: warning: exported method RequestDeniedError.SetDefaults should have comment or be unexported (golint)
  • Line 127: warning: exported method RequestDeniedError.Scan should have comment or be unexported (golint)
  • Line 141: warning: exported method RequestDeniedError.Value should have comment or be unexported (golint)
  • Line 154: warning: comment on exported type RequestWasHandledResponse should be of the form "RequestWasHandledResponse ..." (with optional leading article) (golint)
  • Line 164: warning: comment on exported type HandledConsentRequest should be of the form "HandledConsentRequest ..." (with optional leading article) (golint)
  • Line 206: warning: exported method HandledConsentRequest.TableName should have comment or be unexported (golint)
  • Line 206: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 210: warning: exported method HandledConsentRequest.HasError should have comment or be unexported (golint)
  • Line 214: warning: exported method HandledConsentRequest.BeforeSave should have comment or be unexported (golint)
  • Line 222: warning: exported method HandledConsentRequest.AfterSave should have comment or be unexported (golint)
  • Line 238: warning: exported method HandledConsentRequest.AfterFind should have comment or be unexported (golint)
  • Line 242: warning: comment on exported type PreviousConsentSession should be of the form "PreviousConsentSession ..." (with optional leading article) (golint)
  • Line 344: warning: exported method HandledLoginRequest.TableName should have comment or be unexported (golint)
  • Line 344: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 348: warning: exported method HandledLoginRequest.HasError should have comment or be unexported (golint)
  • Line 352: warning: exported method HandledLoginRequest.AfterUpdate should have comment or be unexported (golint)
  • Line 357: warning: exported method HandledLoginRequest.BeforeSave should have comment or be unexported (golint)
  • Line 364: warning: comment on exported type OpenIDConnectContext should be of the form "OpenIDConnectContext ..." (with optional leading article) (golint)
  • Line 407: warning: exported method OpenIDConnectContext.Scan should have comment or be unexported (golint)
  • Line 415: warning: exported method OpenIDConnectContext.Value should have comment or be unexported (golint)
  • Line 420: warning: comment on exported type LogoutRequest should be of the form "LogoutRequest ..." (with optional leading article) (golint)
  • Line 454: warning: exported method LogoutRequest.TableName should have comment or be unexported (golint)
  • Line 454: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 458: warning: exported method LogoutRequest.BeforeSave should have comment or be unexported (golint)
  • Line 468: warning: exported method LogoutRequest.AfterFind should have comment or be unexported (golint)
  • Line 476: warning: comment on exported type LogoutResult should be of the form "LogoutResult ..." (with optional leading article) (golint)
  • Line 484: warning: comment on exported type LoginRequest should be of the form "LoginRequest ..." (with optional leading article) (golint)
  • Line 557: warning: exported method LoginRequest.TableName should have comment or be unexported (golint)
  • Line 557: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 561: warning: exported method LoginRequest.FindInDB should have comment or be unexported (golint)
  • Line 567: warning: exported method LoginRequest.BeforeSave should have comment or be unexported (golint)
  • Line 574: warning: exported method LoginRequest.AfterFind should have comment or be unexported (golint)
  • Line 579: warning: comment on exported type ConsentRequest should be of the form "ConsentRequest ..." (with optional leading article) (golint)
  • Line 649: warning: exported method ConsentRequest.TableName should have comment or be unexported (golint)
  • Line 649: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 653: warning: exported method ConsentRequest.FindInDB should have comment or be unexported (golint)
  • Line 660: warning: exported method ConsentRequest.BeforeSave should have comment or be unexported (golint)
  • Line 667: warning: exported method ConsentRequest.AfterFind should have comment or be unexported (golint)
  • Line 672: warning: comment on exported type ConsentRequestSessionData should be of the form "ConsentRequestSessionData ..." (with optional leading article) (golint)
  • Line 689: warning: exported function NewConsentRequestSessionData should have comment or be unexported (golint)

• hydra/driver/config/buildinfo.go
  • Line 4: warning: exported var Version should have comment or be unexported (golint)

• hydra/driver/registry_sql.go
  • Line 16: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 31: warning: exported type RegistrySQL should have comment or be unexported (golint)
  • Line 44: warning: exported function NewRegistrySQL should have comment or be unexported (golint)
  • Line 52: warning: exported method RegistrySQL.Init should have comment or be unexported (golint)
  • Line 107: warning: exported method RegistrySQL.Ping should have comment or be unexported (golint)
  • Line 111: warning: exported method RegistrySQL.ClientManager should have comment or be unexported (golint)
  • Line 115: warning: exported method RegistrySQL.ConsentManager should have comment or be unexported (golint)
  • Line 119: warning: exported method RegistrySQL.OAuth2Storage should have comment or be unexported (golint)
  • Line 123: warning: exported method RegistrySQL.KeyManager should have comment or be unexported (golint)

• hydra/jwk/handler.go
  • Line 42: warning: exported const KeyHandlerPath should have comment (or a comment on this block) or be unexported (golint)
  • Line 46: warning: exported type Handler should have comment or be unexported (golint)
  • Line 51: warning: exported function NewHandler should have comment or be unexported (golint)
  • Line 55: warning: exported method Handler.SetRoutes should have comment or be unexported (golint)
  • Line 71: warning: comment on exported method Handler.WellKnown should be of the form "WellKnown ..." (golint)
  • Line 112: warning: comment on exported method Handler.GetKey should be of the form "GetKey ..." (golint)
  • Line 143: warning: comment on exported method Handler.GetKeySet should be of the form "GetKeySet ..." (golint)
  • Line 176: warning: comment on exported method Handler.Create should be of the form "Create ..." (golint)
  • Line 225: warning: comment on exported method Handler.UpdateKeySet should be of the form "UpdateKeySet ..." (golint)
  • Line 268: warning: comment on exported method Handler.UpdateKey should be of the form "UpdateKey ..." (golint)
  • Line 311: warning: comment on exported method Handler.DeleteKeySet should be of the form "DeleteKeySet ..." (golint)
  • Line 343: warning: comment on exported method Handler.DeleteKey should be of the form "DeleteKey ..." (golint)

• hydra/jwk/manager.go
  • Line 31: warning: exported type Manager should have comment or be unexported (golint)
  • Line 45: warning: exported type SQLData should have comment or be unexported (golint)
  • Line 55: warning: exported method SQLData.TableName should have comment or be unexported (golint)

• hydra/cmd/token_client.go
  • Line 56: warning: exported function NewTokenClientCmd should have comment or be unexported (golint)

• hydra/cmd/token_introspect.go
  • Line 31: warning: comment on exported function NewTokenIntrospectCmd should be of the form "NewTokenIntrospectCmd ..." (golint)

• hydra/persistence/sql/persister_client.go
  • Line 15: warning: exported method Persister.GetConcreteClient should have comment or be unexported (golint)
  • Line 20: warning: exported method Persister.GetClient should have comment or be unexported (golint)
  • Line 24: warning: exported method Persister.UpdateClient should have comment or be unexported (golint)
  • Line 47: warning: exported method Persister.Authenticate should have comment or be unexported (golint)
  • Line 60: warning: exported method Persister.CreateClient should have comment or be unexported (golint)
  • Line 70: warning: exported method Persister.DeleteClient should have comment or be unexported (golint)
  • Line 79: warning: exported method Persister.GetClients should have comment or be unexported (golint)
  • Line 88: warning: exported method Persister.CountClients should have comment or be unexported (golint)

• hydra/internal/fizzmigrate/oauth2/sql_migration_files.go
  • Line 720: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests10_testSql should be _oauth2MigrationsSQLTests10TestSQL (golint)
  • Line 722: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests10_testSqlBytes should be oauth2MigrationsSQLTests10TestSQLBytes (golint)
  • Line 729: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests10_testSql should be oauth2MigrationsSQLTests10TestSQL (golint)
  • Line 740: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests11_testSql should be _oauth2MigrationsSQLTests11TestSQL (golint)
  • Line 742: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests11_testSqlBytes should be oauth2MigrationsSQLTests11TestSQLBytes (golint)
  • Line 749: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests11_testSql should be oauth2MigrationsSQLTests11TestSQL (golint)
  • Line 760: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests1_testSql should be _oauth2MigrationsSQLTests1TestSQL (golint)
  • Line 762: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests1_testSqlBytes should be oauth2MigrationsSQLTests1TestSQLBytes (golint)
  • Line 769: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests1_testSql should be oauth2MigrationsSQLTests1TestSQL (golint)
  • Line 780: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests2_testSql should be _oauth2MigrationsSQLTests2TestSQL (golint)
  • Line 782: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests2_testSqlBytes should be oauth2MigrationsSQLTests2TestSQLBytes (golint)
  • Line 789: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests2_testSql should be oauth2MigrationsSQLTests2TestSQL (golint)
  • Line 800: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests3_testSql should be _oauth2MigrationsSQLTests3TestSQL (golint)
  • Line 802: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests3_testSqlBytes should be oauth2MigrationsSQLTests3TestSQLBytes (golint)
  • Line 809: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests3_testSql should be oauth2MigrationsSQLTests3TestSQL (golint)
  • Line 820: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests4_testSql should be _oauth2MigrationsSQLTests4TestSQL (golint)
  • Line 822: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests4_testSqlBytes should be oauth2MigrationsSQLTests4TestSQLBytes (golint)
  • Line 829: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests4_testSql should be oauth2MigrationsSQLTests4TestSQL (golint)
  • Line 840: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests5_testSql should be _oauth2MigrationsSQLTests5TestSQL (golint)
  • Line 842: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests5_testSqlBytes should be oauth2MigrationsSQLTests5TestSQLBytes (golint)
  • Line 849: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests5_testSql should be oauth2MigrationsSQLTests5TestSQL (golint)
  • Line 860: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests6_testSql should be _oauth2MigrationsSQLTests6TestSQL (golint)
  • Line 862: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests6_testSqlBytes should be oauth2MigrationsSQLTests6TestSQLBytes (golint)
  • Line 869: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests6_testSql should be oauth2MigrationsSQLTests6TestSQL (golint)
  • Line 880: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests7_testSql should be _oauth2MigrationsSQLTests7TestSQL (golint)
  • Line 882: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests7_testSqlBytes should be oauth2MigrationsSQLTests7TestSQLBytes (golint)
  • Line 889: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests7_testSql should be oauth2MigrationsSQLTests7TestSQL (golint)
  • Line 900: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests8_testSql should be _oauth2MigrationsSQLTests8TestSQL (golint)
  • Line 902: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests8_testSqlBytes should be oauth2MigrationsSQLTests8TestSQLBytes (golint)
  • Line 909: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests8_testSql should be oauth2MigrationsSQLTests8TestSQL (golint)
  • Line 920: warning: don't use underscores in Go names; var _oauth2MigrationsSqlTests9_testSql should be _oauth2MigrationsSQLTests9TestSQL (golint)
  • Line 922: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests9_testSqlBytes should be oauth2MigrationsSQLTests9TestSQLBytes (golint)
  • Line 929: warning: don't use underscores in Go names; func oauth2MigrationsSqlTests9_testSql should be oauth2MigrationsSQLTests9TestSQL (golint)

• hydra/jwk/generator_hs512.go
  • Line 34: warning: exported type HS512Generator should have comment or be unexported (golint)
  • Line 36: warning: exported method HS512Generator.Generate should have comment or be unexported (golint)

• hydra/oauth2/equalKeys.go
  • Line 31: warning: exported function AssertObjectKeysEqual should have comment or be unexported (golint)
  • Line 42: warning: exported function AssertObjectKeysNotEqual should have comment or be unexported (golint)
  • Line 53: warning: exported function RequireObjectKeysEqual should have comment or be unexported (golint)
  • Line 63: warning: exported function RequireObjectKeysNotEqual should have comment or be unexported (golint)

• hydra/client/validator.go
  • Line 54: warning: exported type Validator should have comment or be unexported (golint)
  • Line 59: warning: exported function NewValidator should have comment or be unexported (golint)
  • Line 66: warning: exported function NewValidatorWithClient should have comment or be unexported (golint)
  • Line 73: warning: exported method Validator.Validate should have comment or be unexported (golint)
  • Line 189: warning: exported method Validator.ValidateSectorIdentifierURL should have comment or be unexported (golint)

• hydra/cmd/migrate_sql.go
  • Line 29: warning: comment on exported function NewMigrateSqlCmd should be of the form "NewMigrateSqlCmd ..." (golint)

• hydra/consent/doc.go
  • Line 144: warning: comment on exported type FlushLoginConsentRequest should be of the form "FlushLoginConsentRequest ..." (with optional leading article) (golint)

• hydra/consent/strategy.go
  • Line 31: warning: exported type Strategy should have comment or be unexported (golint)

• hydra/persistence/sql/persister_jwk.go
  • Line 20: warning: exported method Persister.AddKey should have comment or be unexported (golint)
  • Line 39: warning: exported method Persister.AddKeySet should have comment or be unexported (golint)
  • Line 65: warning: exported method Persister.GetKey should have comment or be unexported (golint)
  • Line 89: warning: exported method Persister.GetKeySet should have comment or be unexported (golint)
  • Line 123: warning: exported method Persister.DeleteKey should have comment or be unexported (golint)
  • Line 127: warning: exported method Persister.DeleteKeySet should have comment or be unexported (golint)

• hydra/driver/registry_nosqlite.go
  • Line 5: warning: exported method RegistrySQL.CanHandle should have comment or be unexported (golint)

• hydra/cmd/token_user.go
  • Line 90: warning: comment on exported function NewTokenUserCmd should be of the form "NewTokenUserCmd ..." (golint)

• hydra/internal/testhelpers/janitor_test_helper.go
  • Line 26: warning: exported type JanitorConsentTestHelper should have comment or be unexported (golint)
  • Line 36: warning: exported function NewConsentJanitorTestHelper should have comment or be unexported (golint)
  • Line 57: warning: exported method JanitorConsentTestHelper.GetDSN should have comment or be unexported (golint)
  • Line 61: warning: exported method JanitorConsentTestHelper.GetConfig should have comment or be unexported (golint)
  • Line 65: warning: exported method JanitorConsentTestHelper.GetNotAfterTestCycles should have comment or be unexported (golint)
  • Line 73: warning: exported method JanitorConsentTestHelper.GetRegistry should have comment or be unexported (golint)
  • Line 78: warning: exported method JanitorConsentTestHelper.AccessTokenNotAfterSetup should have comment or be unexported (golint)
  • Line 89: warning: exported method JanitorConsentTestHelper.AccessTokenNotAfterValidate should have comment or be unexported (golint)
  • Line 108: warning: exported method JanitorConsentTestHelper.RefreshTokenNotAfterSetup should have comment or be unexported (golint)
  • Line 118: warning: exported method JanitorConsentTestHelper.RefreshTokenNotAfterValidate should have comment or be unexported (golint)
  • Line 137: warning: exported method JanitorConsentTestHelper.LoginRejectionSetup should have comment or be unexported (golint)
  • Line 166: warning: exported method JanitorConsentTestHelper.LoginRejectionValidate should have comment or be unexported (golint)
  • Line 181: warning: exported method JanitorConsentTestHelper.ConsentRejectionSetup should have comment or be unexported (golint)
  • Line 212: warning: exported method JanitorConsentTestHelper.ConsentRejectionValidate should have comment or be unexported (golint)
  • Line 227: warning: exported method JanitorConsentTestHelper.LoginTimeoutSetup should have comment or be unexported (golint)
  • Line 249: warning: exported method JanitorConsentTestHelper.LoginTimeoutValidate should have comment or be unexported (golint)
  • Line 265: warning: exported method JanitorConsentTestHelper.ConsentTimeoutSetup should have comment or be unexported (golint)
  • Line 298: warning: exported method JanitorConsentTestHelper.ConsentTimeoutValidate should have comment or be unexported (golint)
  • Line 314: warning: exported method JanitorConsentTestHelper.LoginConsentNotAfterSetup should have comment or be unexported (golint)
  • Line 327: warning: exported method JanitorConsentTestHelper.LoginConsentNotAfterValidate should have comment or be unexported (golint)
  • Line 363: warning: exported method JanitorConsentTestHelper.GetConsentRequestLifespan should have comment or be unexported (golint)
  • Line 367: warning: exported method JanitorConsentTestHelper.GetAccessTokenLifespan should have comment or be unexported (golint)
  • Line 371: warning: exported method JanitorConsentTestHelper.GetRefreshTokenLifespan should have comment or be unexported (golint)
  • Line 392: warning: exported function JanitorTests should have comment or be unexported (golint)

• hydra/x/audit.go
  • Line 9: warning: exported function LogAudit should have comment or be unexported (golint)

• hydra/driver/config/provider.go
  • Line 26: warning: exported const KeyRoot should have comment (or a comment on this block) or be unexported (golint)
  • Line 69: warning: exported const DSNMemory should have comment or be unexported (golint)
  • Line 71: warning: exported type Provider should have comment or be unexported (golint)
  • Line 77: warning: exported function MustNew should have comment or be unexported (golint)
  • Line 85: warning: exported function New should have comment or be unexported (golint)
  • Line 102: warning: exported method Provider.Set should have comment or be unexported (golint)
  • Line 106: warning: exported method Provider.MustSet should have comment or be unexported (golint)
  • Line 112: warning: exported method Provider.Source should have comment or be unexported (golint)
  • Line 116: warning: exported method Provider.InsecureRedirects should have comment or be unexported (golint)
  • Line 120: warning: exported method Provider.WellKnownKeys should have comment or be unexported (golint)
  • Line 129: warning: exported method Provider.IsUsingJWTAsAccessTokens should have comment or be unexported (golint)
  • Line 133: warning: exported method Provider.SubjectTypesSupported should have comment or be unexported (golint)
  • Line 161: warning: exported method Provider.DefaultClientScope should have comment or be unexported (golint)
  • Line 168: warning: exported method Provider.DSN should have comment or be unexported (golint)
  • Line 183: warning: exported method Provider.EncryptSessionData should have comment or be unexported (golint)
  • Line 187: warning: exported method Provider.ExcludeNotBeforeClaim should have comment or be unexported (golint)
  • Line 191: warning: exported method Provider.DataSourcePlugin should have comment or be unexported (golint)
  • Line 195: warning: exported method Provider.BCryptCost should have comment or be unexported (golint)
  • Line 199: warning: exported method Provider.CookieSameSiteMode should have comment or be unexported (golint)
  • Line 219: warning: exported method Provider.CookieSameSiteLegacyWorkaround should have comment or be unexported (golint)
  • Line 223: warning: exported method Provider.ConsentRequestMaxAge should have comment or be unexported (golint)
  • Line 227: warning: exported method Provider.AccessTokenLifespan should have comment or be unexported (golint)
  • Line 231: warning: exported method Provider.RefreshTokenLifespan should have comment or be unexported (golint)
  • Line 235: warning: exported method Provider.IDTokenLifespan should have comment or be unexported (golint)
  • Line 239: warning: exported method Provider.AuthCodeLifespan should have comment or be unexported (golint)
  • Line 243: warning: exported method Provider.ScopeStrategy should have comment or be unexported (golint)
  • Line 247: warning: exported method Provider.Tracing should have comment or be unexported (golint)
  • Line 251: warning: exported method Provider.GetCookieSecrets should have comment or be unexported (golint)
  • Line 264: warning: exported method Provider.GetRotatedSystemSecrets should have comment or be unexported (golint)
  • Line 279: warning: exported method Provider.GetSystemSecret should have comment or be unexported (golint)
  • Line 307: warning: exported method Provider.LogoutRedirectURL should have comment or be unexported (golint)
  • Line 331: warning: exported method Provider.LoginURL should have comment or be unexported (golint)
  • Line 335: warning: exported method Provider.LogoutURL should have comment or be unexported (golint)
  • Line 339: warning: exported method Provider.ConsentURL should have comment or be unexported (golint)
  • Line 343: warning: exported method Provider.ErrorURL should have comment or be unexported (golint)
  • Line 347: warning: exported method Provider.PublicURL should have comment or be unexported (golint)
  • Line 351: warning: exported method Provider.IssuerURL should have comment or be unexported (golint)
  • Line 357: warning: exported method Provider.OAuth2ClientRegistrationURL should have comment or be unexported (golint)
  • Line 361: warning: exported method Provider.OAuth2TokenURL should have comment or be unexported (golint)
  • Line 365: warning: exported method Provider.OAuth2AuthURL should have comment or be unexported (golint)
  • Line 369: warning: exported method Provider.JWKSURL should have comment or be unexported (golint)
  • Line 373: warning: exported method Provider.AccessTokenStrategy should have comment or be unexported (golint)
  • Line 377: warning: exported method Provider.SubjectIdentifierAlgorithmSalt should have comment or be unexported (golint)
  • Line 381: warning: exported method Provider.OIDCDiscoverySupportedClaims should have comment or be unexported (golint)
  • Line 390: warning: exported method Provider.OIDCDiscoverySupportedScope should have comment or be unexported (golint)
  • Line 399: warning: exported method Provider.OIDCDiscoveryUserinfoEndpoint should have comment or be unexported (golint)
  • Line 403: warning: exported method Provider.ShareOAuth2Debug should have comment or be unexported (golint)
  • Line 407: warning: exported method Provider.OAuth2LegacyErrors should have comment or be unexported (golint)
  • Line 411: warning: exported method Provider.PKCEEnforced should have comment or be unexported (golint)
  • Line 415: warning: exported method Provider.EnforcePKCEForPublicClients should have comment or be unexported (golint)
  • Line 419: warning: exported method Provider.CGroupsV1AutoMaxProcsEnabled should have comment or be unexported (golint)
  • Line 423: warning: exported method Provider.GrantAllClientCredentialsScopesPerDefault should have comment or be unexported (golint)

• hydra/oauth2/doc.go
  • Line 161: warning: comment on exported type FlushInactiveOAuth2TokensRequest should be of the form "FlushInactiveOAuth2TokensRequest ..." (with optional leading article) (golint)

• hydra/client/handler.go
  • Line 43: warning: exported type Handler should have comment or be unexported (golint)
  • Line 48: warning: exported const ClientsHandlerPath should have comment (or a comment on this block) or be unexported (golint)
  • Line 51: warning: exported function NewHandler should have comment or be unexported (golint)
  • Line 57: warning: exported method Handler.SetRoutes should have comment or be unexported (golint)
  • Line 66: warning: comment on exported method Handler.Create should be of the form "Create ..." (golint)
  • Line 125: warning: comment on exported method Handler.Update should be of the form "Update ..." (golint)
  • Line 161: warning: comment on exported method Handler.Patch should be of the form "Patch ..." (golint)
  • Line 224: warning: comment on exported method Handler.List should be of the form "List ..." (golint)
  • Line 273: warning: comment on exported method Handler.Get should be of the form "Get ..." (golint)
  • Line 310: warning: comment on exported method Handler.Delete should be of the form "Delete ..." (golint)

• hydra/cmd/token_flush.go
  • Line 32: warning: comment on exported function NewTokenFlushCmd should be of the form "NewTokenFlushCmd ..." (golint)

• hydra/x/json.go
  • Line 10: warning: exported function ApplyJSONPatch should have comment or be unexported (golint)

• hydra/internal/fizzmigrate/jwk/sql_migration_files.go
  • Line 293: warning: don't use underscores in Go names; var _jwkMigrationsSqlTests1_testSql should be _jwkMigrationsSQLTests1TestSQL (golint)
  • Line 295: warning: don't use underscores in Go names; func jwkMigrationsSqlTests1_testSqlBytes should be jwkMigrationsSQLTests1TestSQLBytes (golint)
  • Line 302: warning: don't use underscores in Go names; func jwkMigrationsSqlTests1_testSql should be jwkMigrationsSQLTests1TestSQL (golint)
  • Line 313: warning: don't use underscores in Go names; var _jwkMigrationsSqlTests2_testSql should be _jwkMigrationsSQLTests2TestSQL (golint)
  • Line 315: warning: don't use underscores in Go names; func jwkMigrationsSqlTests2_testSqlBytes should be jwkMigrationsSQLTests2TestSQLBytes (golint)
  • Line 322: warning: don't use underscores in Go names; func jwkMigrationsSqlTests2_testSql should be jwkMigrationsSQLTests2TestSQL (golint)
  • Line 333: warning: don't use underscores in Go names; var _jwkMigrationsSqlTests3_testSql should be _jwkMigrationsSQLTests3TestSQL (golint)
  • Line 335: warning: don't use underscores in Go names; func jwkMigrationsSqlTests3_testSqlBytes should be jwkMigrationsSQLTests3TestSQLBytes (golint)
  • Line 342: warning: don't use underscores in Go names; func jwkMigrationsSqlTests3_testSql should be jwkMigrationsSQLTests3TestSQL (golint)
  • Line 353: warning: don't use underscores in Go names; var _jwkMigrationsSqlTests4_testSql should be _jwkMigrationsSQLTests4TestSQL (golint)
  • Line 355: warning: don't use underscores in Go names; func jwkMigrationsSqlTests4_testSqlBytes should be jwkMigrationsSQLTests4TestSQLBytes (golint)
  • Line 362: warning: don't use underscores in Go names; func jwkMigrationsSqlTests4_testSql should be jwkMigrationsSQLTests4TestSQL (golint)

• hydra/internal/testhelpers/oauth2.go
  • Line 33: warning: exported function NewIDToken should have comment or be unexported (golint)
  • Line 37: warning: exported function NewIDTokenWithExpiry should have comment or be unexported (golint)
  • Line 47: warning: exported function NewIDTokenWithClaims should have comment or be unexported (golint)
  • Line 53: warning: exported function NewOAuth2Server should have comment or be unexported (golint)
  • Line 78: warning: exported function DecodeIDToken should have comment or be unexported (golint)
  • Line 89: warning: exported function IntrospectToken should have comment or be unexported (golint)
  • Line 103: warning: exported function Userinfo should have comment or be unexported (golint)
  • Line 116: warning: exported function HTTPServerNotImplementedHandler should have comment or be unexported (golint)
  • Line 120: warning: exported function HTTPServerNoExpectedCallHandler should have comment or be unexported (golint)
  • Line 126: warning: exported function NewLoginConsentUI should have comment or be unexported (golint)
  • Line 145: warning: exported function NewCallbackURL should have comment or be unexported (golint)
  • Line 160: warning: exported function NewEmptyCookieJar should have comment or be unexported (golint)
  • Line 166: warning: exported function NewEmptyJarClient should have comment or be unexported (golint)

• hydra/driver/registry.go
  • Line 30: warning: exported type Registry should have comment or be unexported (golint)
  • Line 62: warning: exported function NewRegistryFromDSN should have comment or be unexported (golint)
  • Line 82: warning: exported function CallRegistry should have comment or be unexported (golint)

• hydra/x/fosite_storer.go
  • Line 33: warning: exported type FositeStorer should have comment or be unexported (golint)

• hydra/consent/registry.go
  • Line 11: warning: exported type InternalRegistry should have comment or be unexported (golint)
  • Line 24: warning: exported type Registry should have comment or be unexported (golint)

• hydra/internal/fosite_store.go
  • Line 10: warning: exported function AddFositeExamples should have comment or be unexported (golint)

• hydra/jwk/cast.go
  • Line 30: warning: exported function MustRSAPublic should have comment or be unexported (golint)
  • Line 39: warning: exported function ToRSAPublic should have comment or be unexported (golint)
  • Line 48: warning: exported function MustRSAPrivate should have comment or be unexported (golint)
  • Line 57: warning: exported function ToRSAPrivate should have comment or be unexported (golint)

• hydra/x/hasher.go
  • Line 1: warning: package comment should be of the form "Package x ..." (golint)
  • Line 1: warning: package comment should not have leading space (golint)
  • Line 49: warning: exported method BCrypt.Hash should have comment or be unexported (golint)
  • Line 61: warning: exported method BCrypt.Compare should have comment or be unexported (golint)

• hydra/internal/fizzmigrate/oauth2/migrator.go
  • Line 14: warning: exported var Migrations should have comment or be unexported (golint)
  • Line 32: warning: exported function NewMigrator should have comment or be unexported (golint)
  • Line 51: warning: receiver name m should be consistent with previous receiver name s for migrator (golint)

• hydra/cmd/clients_update.go
  • Line 9: warning: comment on exported function NewClientsUpdateCmd should be of the form "NewClientsUpdateCmd ..." (golint)

• hydra/cmd/serve_all.go
  • Line 23: warning: comment on exported function NewServeAllCmd should be of the form "NewServeAllCmd ..." (golint)

• hydra/persistence/sql/persister.go
  • Line 28: warning: exported var ErrTransactionOpen should have comment or be unexported (golint)
  • Line 33: warning: exported type Persister should have comment or be unexported (golint)
  • Line 40: warning: exported type Dependencies should have comment or be unexported (golint)
  • Line 49: warning: exported method Persister.BeginTX should have comment or be unexported (golint)
  • Line 68: warning: exported method Persister.Commit should have comment or be unexported (golint)
  • Line 77: warning: exported method Persister.Rollback should have comment or be unexported (golint)
  • Line 86: warning: exported function NewPersister should have comment or be unexported (golint)
  • Line 101: warning: exported method Persister.Connection should have comment or be unexported (golint)

• hydra/x/registry.go
  • Line 14: warning: exported type RegistryLogger should have comment or be unexported (golint)
  • Line 19: warning: exported type RegistryWriter should have comment or be unexported (golint)
  • Line 23: warning: exported type RegistryCookieStore should have comment or be unexported (golint)
  • Line 27: warning: exported type TracingProvider should have comment or be unexported (golint)

• hydra/jwk/helper.go
  • Line 40: warning: exported function EnsureAsymmetricKeypairExists should have comment or be unexported (golint)
  • Line 45: warning: exported function AsymmetricKeypair should have comment or be unexported (golint)
  • Line 59: warning: exported function GetOrCreateKey should have comment or be unexported (golint)
  • Line 107: warning: exported function First should have comment or be unexported (golint)
  • Line 114: warning: exported function FindKeyByPrefix should have comment or be unexported (golint)
  • Line 123: warning: exported function FindKeysByPrefix should have comment or be unexported (golint)
  • Line 139: warning: exported function PEMBlockForKey should have comment or be unexported (golint)
  • Line 154: warning: exported function Ider should have comment or be unexported (golint)

• hydra/client/registry.go
  • Line 8: warning: exported type InternalRegistry should have comment or be unexported (golint)
  • Line 13: warning: exported type Registry should have comment or be unexported (golint)

• hydra/internal/fizzmigrate/consent/migrator.go
  • Line 14: warning: exported var Migrations should have comment or be unexported (golint)
  • Line 32: warning: exported function NewMigrator should have comment or be unexported (golint)

• hydra/cmd/clients.go
  • Line 30: warning: comment on exported function NewClientsCmd should be of the form "NewClientsCmd ..." (golint)

• hydra/cmd/token_revoke.go
  • Line 31: warning: comment on exported function NewTokenRevokeCmd should be of the form "NewTokenRevokeCmd ..." (golint)

• hydra/cmd/cli/handler_token.go
  • Line 41: warning: exported type TokenHandler should have comment or be unexported (golint)
  • Line 47: warning: exported method TokenHandler.RevokeToken should have comment or be unexported (golint)
  • Line 67: warning: exported method TokenHandler.FlushTokens should have comment or be unexported (golint)
  • Line 76: warning: exported method TokenHandler.DeleteToken should have comment or be unexported (golint)

• hydra/x/oauth2cors/cors.go
  • Line 41: warning: exported function Middleware should have comment or be unexported (golint)

• hydra/internal/driver.go
  • Line 30: warning: exported function NewConfigurationWithDefaults should have comment or be unexported (golint)
  • Line 37: warning: exported function NewConfigurationWithDefaultsAndHTTPS should have comment or be unexported (golint)
  • Line 44: warning: exported function NewRegistryMemory should have comment or be unexported (golint)
  • Line 48: warning: exported function NewMockedRegistry should have comment or be unexported (golint)
  • Line 52: warning: exported function NewRegistrySQLFromURL should have comment or be unexported (golint)
  • Line 67: warning: exported function CleanAndMigrate should have comment or be unexported (golint)
  • Line 75: warning: exported function ConnectToMySQL should have comment or be unexported (golint)
  • Line 85: warning: exported function ConnectToPG should have comment or be unexported (golint)
  • Line 91: warning: exported function ConnectToCRDB should have comment or be unexported (golint)
  • Line 101: warning: exported function ConnectDatabases should have comment or be unexported (golint)
  • Line 146: warning: exported function MustEnsureRegistryKeys should have comment or be unexported (golint)

• hydra/persistence/sql/persister_migration.go
  • Line 23: warning: exported method Persister.MigrationStatus should have comment or be unexported (golint)
  • Line 32: warning: exported method Persister.MigrateDown should have comment or be unexported (golint)
  • Line 36: warning: exported method Persister.MigrateUp should have comment or be unexported (golint)
  • Line 43: warning: exported method Persister.MigrateUpTo should have comment or be unexported (golint)
  • Line 51: warning: exported method Persister.PrepareMigration should have comment or be unexported (golint)
  • Line 64: warning: comment on exported type OldMigrationRecord should be of the form "OldMigrationRecord ..." (with optional leading article) (golint)

• hydra/jwk/jwt_strategy.go
  • Line 37: warning: exported type JWTStrategy should have comment or be unexported (golint)
  • Line 43: warning: exported type RS256JWTStrategy should have comment or be unexported (golint)
  • Line 57: warning: exported function NewRS256JWTStrategy should have comment or be unexported (golint)
  • Line 65: warning: exported method RS256JWTStrategy.Hash should have comment or be unexported (golint)
  • Line 74: warning: exported method RS256JWTStrategy.GetSignature should have comment or be unexported (golint)
  • Line 78: warning: exported method RS256JWTStrategy.Generate should have comment or be unexported (golint)
  • Line 86: warning: exported method RS256JWTStrategy.Validate should have comment or be unexported (golint)
  • Line 94: warning: exported method RS256JWTStrategy.Decode should have comment or be unexported (golint)
  • Line 102: warning: exported method RS256JWTStrategy.GetPublicKeyID should have comment or be unexported (golint)
  • Line 132: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 141: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)

• hydra/cmd/keys_create.go
  • Line 29: warning: comment on exported function NewKeysCreateCmd should be of the form "NewKeysCreateCmd ..." (golint)

• hydra/spec/config.go
  • Line 3: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 5: warning: comment on exported var ConfigValidationSchema should be of the form "ConfigValidationSchema ..." (golint)

• hydra/cmd/migrate.go
  • Line 29: warning: comment on exported function NewMigrateCmd should be of the form "NewMigrateCmd ..." (golint)

• hydra/internal/testhelpers/server.go
  • Line 9: warning: exported function FlexibleServer should have comment or be unexported (golint)

• hydra/driver/config/tls.go
  • Line 10: warning: exported const KeySuffixTLSEnabled should have comment (or a comment on this block) or be unexported (golint)
  • Line 24: warning: exported type TLSConfig should have comment or be unexported (golint)
  • Line 30: warning: exported method Provider.TLS should have comment or be unexported (golint)

• hydra/cmd/keys_get.go
  • Line 29: warning: comment on exported function NewKeysGetCmd should be of the form "NewKeysGetCmd ..." (golint)

• hydra/cmd/cli/handler_migrate.go
  • Line 21: warning: exported type MigrateHandler should have comment or be unexported (golint)
  • Line 27: warning: exported method MigrateHandler.MigrateSQL should have comment or be unexported (golint)

• hydra/internal/fizzmigrate/jwk/migrator.go
  • Line 14: warning: exported var Migrations should have comment or be unexported (golint)
  • Line 32: warning: exported function NewMigrator should have comment or be unexported (golint)

• hydra/oauth2/registry.go
  • Line 12: warning: exported type InternalRegistry should have comment or be unexported (golint)
  • Line 20: warning: exported type Registry should have comment or be unexported (golint)

• hydra/cmd/server/handler.go
  • Line 60: warning: exported function EnhanceMiddleware should have comment or be unexported (golint)
  • Line 90: warning: exported function RunServeAdmin should have comment or be unexported (golint)
  • Line 116: warning: exported function RunServePublic should have comment or be unexported (golint)
  • Line 142: warning: exported function RunServeAll should have comment or be unexported (golint)
  • Line 328: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• hydra/cmd/server/helper_cert.go
  • Line 48: warning: exported function AttachCertificate should have comment or be unexported (golint)
  • Line 57: warning: exported function GetOrCreateTLSCertificate should have comment or be unexported (golint)

• hydra/cmd/clients_get.go
  • Line 29: warning: exported function NewClientsGetCmd should have comment or be unexported (golint)

• hydra/cmd/version.go
  • Line 31: warning: comment on exported function NewVersionCmd should be of the form "NewVersionCmd ..." (golint)

• hydra/x/errors.go
  • Line 31: warning: exported var ErrNotFound should have comment or be unexported (golint)
  • Line 43: warning: exported function LogError should have comment or be unexported (golint)

• hydra/cmd/clients_create.go
  • Line 29: warning: comment on exported function NewClientsCreateCmd should be of the form "NewClientsCreateCmd ..." (golint)

• hydra/x/sqlx.go
  • Line 13: warning: comment on exported type JoseJSONWebKeySet should be of the form "JoseJSONWebKeySet ..." (with optional leading article) (golint)
  • Line 19: warning: exported method JoseJSONWebKeySet.Scan should have comment or be unexported (golint)
  • Line 27: warning: exported method JoseJSONWebKeySet.Value should have comment or be unexported (golint)

• hydra/cmd/serve.go
  • Line 38: warning: comment on exported function NewServeCmd should be of the form "NewServeCmd ..." (golint)

• hydra/cmd/token.go
  • Line 29: warning: comment on exported function NewTokenCmd should be of the form "NewTokenCmd ..." (golint)

• hydra/consent/janitor_consent_test_helper.go
  • Line 9: warning: exported function NewHandledLoginRequest should have comment or be unexported (golint)
  • Line 31: warning: exported function NewHandledConsentRequest should have comment or be unexported (golint)

• hydra/consent/subject_identifier_algorithm.go
  • Line 25: warning: exported type SubjectIdentifierAlgorithm should have comment or be unexported (golint)

• hydra/consent/subject_identifier_algorithm_public.go
  • Line 25: warning: exported type SubjectIdentifierAlgorithmPublic should have comment or be unexported (golint)
  • Line 27: warning: exported function NewSubjectIdentifierAlgorithmPublic should have comment or be unexported (golint)
  • Line 31: warning: exported method SubjectIdentifierAlgorithmPublic.Obfuscate should have comment or be unexported (golint)

• hydra/cmd/clients_import.go
  • Line 29: warning: exported function NewClientsImportCmd should have comment or be unexported (golint)

• hydra/driver/factory.go
  • Line 28: warning: exported type OptionsModifier should have comment or be unexported (golint)
  • Line 30: warning: exported function WithOptions should have comment or be unexported (golint)
  • Line 45: warning: comment on exported function DisablePreloading should be of the form "DisablePreloading ..." (golint)
  • Line 54: warning: exported function New should have comment or be unexported (golint)

• hydra/internal/fizzmigrate/client/migrator.go
  • Line 14: warning: exported var Migrations should have comment or be unexported (golint)
  • Line 24: warning: exported function NewMigrator should have comment or be unexported (golint)

• hydra/oauth2/handler.go
  • Line 52: warning: exported const DefaultLoginPath should have comment (or a comment on this block) or be unexported (golint)
  • Line 72: warning: exported type Handler should have comment or be unexported (golint)
  • Line 77: warning: exported function NewHandler should have comment or be unexported (golint)
  • Line 81: warning: exported method Handler.SetRoutes should have comment or be unexported (golint)
  • Line 114: warning: comment on exported method Handler.LogoutHandler should be of the form "LogoutHandler ..." (golint)
  • Line 207: warning: comment on exported method Handler.WellKnownHandler should be of the form "WellKnownHandler ..." (golint)
  • Line 257: warning: comment on exported method Handler.UserinfoHandler should be of the form "UserinfoHandler ..." (golint)
  • Line 360: warning: comment on exported method Handler.RevocationHandler should be of the form "RevocationHandler ..." (golint)
  • Line 393: warning: comment on exported method Handler.IntrospectHandler should be of the form "IntrospectHandler ..." (golint)
  • Line 504: warning: comment on exported method Handler.FlushHandler should be of the form "FlushHandler ..." (golint)
  • Line 545: warning: comment on exported method Handler.TokenHandler should be of the form "TokenHandler ..." (golint)
  • Line 646: warning: comment on exported method Handler.AuthHandler should be of the form "AuthHandler ..." (golint)
  • Line 774: warning: comment on exported method Handler.DeleteHandler should be of the form "DeleteHandler ..." (golint)

• hydra/client/doc.go
  • Line 29: warning: package comment is detached; there should be no blank lines between it and the package statement (golint)

• hydra/cmd/keys_delete.go
  • Line 29: warning: comment on exported function NewKeysDeleteCmd should be of the form "NewKeysDeleteCmd ..." (golint)

• hydra/consent/manager_test_helpers.go
  • Line 42: warning: exported function MockConsentRequest should have comment or be unexported (golint)
  • Line 102: warning: exported function MockLogoutRequest should have comment or be unexported (golint)
  • Line 123: warning: exported function MockAuthRequest should have comment or be unexported (golint)
  • Line 173: warning: exported function SaneMockHandleConsentRequest should have comment or be unexported (golint)
  • Line 261: warning: exported function ManagerTests should have comment or be unexported (golint)

• hydra/persistence/definitions.go
  • Line 16: warning: exported type Persister should have comment or be unexported (golint)
  • Line 28: warning: exported type Provider should have comment or be unexported (golint)

• hydra/jwk/generator_hs256.go
  • Line 34: warning: exported type HS256Generator should have comment or be unexported (golint)
  • Line 36: warning: exported method HS256Generator.Generate should have comment or be unexported (golint)

• hydra/cmd/root.go
  • Line 30: warning: comment on exported function NewRootCmd should be of the form "NewRootCmd ..." (golint)
  • Line 40: warning: exported function RegisterCommandRecursive should have comment or be unexported (golint)

• hydra/cmd/cli/handler_janitor.go
  • Line 23: warning: exported const KeepIfYounger should have comment (or a comment on this block) or be unexported (golint)
  • Line 33: warning: exported type JanitorHandler should have comment or be unexported (golint)
  • Line 35: warning: exported function NewJanitorHandler should have comment or be unexported (golint)
  • Line 39: warning: exported method JanitorHandler.Args should have comment or be unexported (golint)
  • Line 39: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 59: warning: exported method JanitorHandler.RunE should have comment or be unexported (golint)
  • Line 59: warning: receiver name should not be an underscore, omit the name if it is unused (golint)

• hydra/x/swagger.go
  • Line 3: warning: comment on exported type JSONWebKey should be of the form "JSONWebKey ..." (with optional leading article) (golint)

• hydra/jwk/manager_test_helpers.go
  • Line 37: warning: exported function RandomBytes should have comment or be unexported (golint)
  • Line 58: warning: exported function TestHelperManagerKey should have comment or be unexported (golint)
  • Line 111: warning: exported function TestHelperManagerKeySet should have comment or be unexported (golint)

• hydra/cmd/janitor.go
  • Line 10: warning: exported function NewJanitorCmd should have comment or be unexported (golint)

• hydra/cmd/serve_admin.go
  • Line 23: warning: comment on exported function NewServeAdminCmd should be of the form "NewServeAdminCmd ..." (golint)

• hydra/cmd/cli/handler_client.go
  • Line 39: warning: exported type ClientHandler should have comment or be unexported (golint)
  • Line 45: warning: exported method ClientHandler.ImportClients should have comment or be unexported (golint)
  • Line 84: warning: exported method ClientHandler.CreateClient should have comment or be unexported (golint)
  • Line 154: warning: exported method ClientHandler.UpdateClient should have comment or be unexported (golint)
  • Line 214: warning: exported method ClientHandler.DeleteClient should have comment or be unexported (golint)
  • Line 226: warning: exported method ClientHandler.GetClient should have comment or be unexported (golint)
  • Line 240: warning: exported method ClientHandler.ListClients should have comment or be unexported (golint)

• hydra/spec/api.go
  • Line 3: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 5: warning: comment on exported var API should be of the form "API ..." (golint)

• hydra/oauth2/handler_fallback_endpoints.go
  • Line 78: warning: exported method Handler.DefaultErrorHandler should have comment or be unexported (golint)

• hydra/client/manager_test_helpers.go
  • Line 38: warning: exported function TestHelperClientAutoGenerateKey should have comment or be unexported (golint)
  • Line 53: warning: exported function TestHelperClientAuthenticate should have comment or be unexported (golint)
  • Line 71: warning: exported function TestHelperUpdateTwoClients should have comment or be unexported (golint)
  • Line 85: warning: exported function TestHelperCreateGetUpdateDeleteClient should have comment or be unexported (golint)

• hydra/cmd/keys_import.go
  • Line 23: warning: comment on exported function NewKeysImportCmd should be of the form "NewKeysImportCmd ..." (golint)

• hydra/cmd/token_delete.go
  • Line 31: warning: comment on exported function NewTokenDeleteCmd should be of the form "NewTokenDeleteCmd ..." (golint)

• hydra/consent/strategy_default.go
  • Line 61: warning: exported const CookieAuthenticationName should have comment (or a comment on this block) or be unexported (golint)
  • Line 68: warning: exported type DefaultStrategy should have comment or be unexported (golint)
  • Line 73: warning: exported function NewStrategy should have comment or be unexported (golint)
  • Line 83: warning: exported var ErrAbortOAuth2Request should have comment or be unexported (golint)
  • Line 84: warning: exported var ErrNoPreviousConsentFound should have comment or be unexported (golint)
  • Line 85: warning: exported var ErrNoAuthenticationSessionFound should have comment or be unexported (golint)
  • Line 86: warning: exported var ErrHintDoesNotMatchAuthentication should have comment or be unexported (golint)
  • Line 997: warning: exported method DefaultStrategy.HandleOpenIDConnectLogout should have comment or be unexported (golint)
  • Line 1006: warning: exported method DefaultStrategy.HandleOAuth2AuthorizationRequest should have comment or be unexported (golint)

• hydra/jwk/registry.go
  • Line 7: warning: exported type InternalRegistry should have comment or be unexported (golint)
  • Line 13: warning: exported type Registry should have comment or be unexported (golint)

• hydra/jwk/generator_rs256.go
  • Line 33: warning: exported type RS256Generator should have comment or be unexported (golint)
  • Line 37: warning: exported method RS256Generator.Generate should have comment or be unexported (golint)

• hydra/oauth2/session.go
  • Line 33: warning: exported type Session should have comment or be unexported (golint)
  • Line 42: warning: exported function NewSession should have comment or be unexported (golint)
  • Line 53: warning: exported method Session.GetJWTClaims should have comment or be unexported (golint)
  • Line 86: warning: exported method Session.GetJWTHeader should have comment or be unexported (golint)
  • Line 92: warning: exported method Session.Clone should have comment or be unexported (golint)

• hydra/client/manager.go
  • Line 29: warning: exported type Manager should have comment or be unexported (golint)
  • Line 35: warning: exported type Storage should have comment or be unexported (golint)

• hydra/driver/config/serve.go
  • Line 14: warning: exported const KeySuffixListenOnHost should have comment (or a comment on this block) or be unexported (golint)
  • Line 23: warning: exported var PublicInterface should have comment or be unexported (golint)
  • Line 31: warning: exported type ServeInterface should have comment or be unexported (golint)
  • Line 46: warning: exported method Provider.ListenOn should have comment or be unexported (golint)
  • Line 54: warning: exported method Provider.SocketPermission should have comment or be unexported (golint)
  • Line 62: warning: exported method Provider.CORS should have comment or be unexported (golint)
  • Line 71: warning: exported method Provider.DisableHealthAccessLog should have comment or be unexported (golint)

• hydra/consent/helper.go
  • Line 107: warning: exported function CookieName should have comment or be unexported (golint)

• hydra/consent/manager.go
  • Line 30: warning: exported type ForcedObfuscatedLoginSession should have comment or be unexported (golint)
  • Line 36: warning: exported method ForcedObfuscatedLoginSession.TableName should have comment or be unexported (golint)
  • Line 36: warning: receiver name should not be an underscore, omit the name if it is unused (golint)
  • Line 40: warning: exported type Manager should have comment or be unexported (golint)

• hydra/cmd/cli/handler_introspection.go
  • Line 38: warning: exported type IntrospectionHandler should have comment or be unexported (golint)
  • Line 44: warning: exported method IntrospectionHandler.Introspect should have comment or be unexported (golint)

• hydra/consent/subject_identifier_algorithm_pairwise.go
  • Line 34: warning: exported type SubjectIdentifierAlgorithmPairwise should have comment or be unexported (golint)
  • Line 38: warning: exported function NewSubjectIdentifierAlgorithmPairwise should have comment or be unexported (golint)
  • Line 42: warning: exported method SubjectIdentifierAlgorithmPairwise.Obfuscate should have comment or be unexported (golint)

• hydra/x/secret.go
  • Line 31: warning: exported function GenerateSecret should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign98%

IneffAssign detects ineffectual assignments in Go code.

• hydra/jwk/helper_test.go
  • Line 64: warning: ineffectual assignment to jwks (ineffassign)

• hydra/client/manager_test_helpers.go
  • Line 157: warning: ineffectual assignment to ds (ineffassign)

• hydra/oauth2/handler_test.go
  • Line 372: warning: ineffectual assignment to err (ineffassign)

misspell99%

Misspell Finds commonly misspelled English words

• hydra/client/handler.go
  • Line 70: warning: "somwhere" is a misspelling of "somewhere" (misspell)

• hydra/oauth2/oauth2_auth_code_test.go
  • Line 235: warning: "strat" is a misspelling of "start" (misspell)
  • Line 238: warning: "strat" is a misspelling of "start" (misspell)
  • Line 583: warning: "strat" is a misspelling of "start" (misspell)