ory-am/hydra

View on GitHub
internal/httpclient/api_oidc.go

Summary

Maintainability
F
1 wk
Test Coverage

File api_oidc.go has 666 lines of code (exceeds 500 allowed). Consider refactoring.
Open

/*
Ory Hydra API

Documentation for all of Ory Hydra's APIs.

Severity: Minor
Found in internal/httpclient/api_oidc.go - About 6 hrs to fix

    Method OidcAPIService.SetOidcDynamicClientExecute has 78 lines of code (exceeds 50 allowed). Consider refactoring.
    Open

    func (a *OidcAPIService) SetOidcDynamicClientExecute(r ApiSetOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
        var (
            localVarHTTPMethod  = http.MethodPut
            localVarPostBody    interface{}
            formFiles           []formFile
    Severity: Major
    Found in internal/httpclient/api_oidc.go - About 2 hrs to fix

      Method OidcAPIService.CreateOidcDynamicClientExecute has 77 lines of code (exceeds 50 allowed). Consider refactoring.
      Open

      func (a *OidcAPIService) CreateOidcDynamicClientExecute(r ApiCreateOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
          var (
              localVarHTTPMethod  = http.MethodPost
              localVarPostBody    interface{}
              formFiles           []formFile
      Severity: Major
      Found in internal/httpclient/api_oidc.go - About 2 hrs to fix

        Method OidcAPIService.CreateVerifiableCredentialExecute has 74 lines of code (exceeds 50 allowed). Consider refactoring.
        Open

        func (a *OidcAPIService) CreateVerifiableCredentialExecute(r ApiCreateVerifiableCredentialRequest) (*VerifiableCredentialResponse, *http.Response, error) {
            var (
                localVarHTTPMethod  = http.MethodPost
                localVarPostBody    interface{}
                formFiles           []formFile
        Severity: Minor
        Found in internal/httpclient/api_oidc.go - About 1 hr to fix

          Method OidcAPIService.GetOidcDynamicClientExecute has 63 lines of code (exceeds 50 allowed). Consider refactoring.
          Open

          func (a *OidcAPIService) GetOidcDynamicClientExecute(r ApiGetOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
              var (
                  localVarHTTPMethod  = http.MethodGet
                  localVarPostBody    interface{}
                  formFiles           []formFile
          Severity: Minor
          Found in internal/httpclient/api_oidc.go - About 1 hr to fix

            Method OidcAPIService.DiscoverOidcConfigurationExecute has 62 lines of code (exceeds 50 allowed). Consider refactoring.
            Open

            func (a *OidcAPIService) DiscoverOidcConfigurationExecute(r ApiDiscoverOidcConfigurationRequest) (*OidcConfiguration, *http.Response, error) {
                var (
                    localVarHTTPMethod  = http.MethodGet
                    localVarPostBody    interface{}
                    formFiles           []formFile
            Severity: Minor
            Found in internal/httpclient/api_oidc.go - About 1 hr to fix

              Method OidcAPIService.GetOidcUserInfoExecute has 62 lines of code (exceeds 50 allowed). Consider refactoring.
              Open

              func (a *OidcAPIService) GetOidcUserInfoExecute(r ApiGetOidcUserInfoRequest) (*OidcUserInfo, *http.Response, error) {
                  var (
                      localVarHTTPMethod  = http.MethodGet
                      localVarPostBody    interface{}
                      formFiles           []formFile
              Severity: Minor
              Found in internal/httpclient/api_oidc.go - About 1 hr to fix

                Method OidcAPIService.DeleteOidcDynamicClientExecute has 54 lines of code (exceeds 50 allowed). Consider refactoring.
                Open

                func (a *OidcAPIService) DeleteOidcDynamicClientExecute(r ApiDeleteOidcDynamicClientRequest) (*http.Response, error) {
                    var (
                        localVarHTTPMethod = http.MethodDelete
                        localVarPostBody   interface{}
                        formFiles          []formFile
                Severity: Minor
                Found in internal/httpclient/api_oidc.go - About 1 hr to fix

                  Method OidcAPIService.SetOidcDynamicClientExecute has 11 return statements (exceeds 4 allowed).
                  Open

                  func (a *OidcAPIService) SetOidcDynamicClientExecute(r ApiSetOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
                      var (
                          localVarHTTPMethod  = http.MethodPut
                          localVarPostBody    interface{}
                          formFiles           []formFile
                  Severity: Major
                  Found in internal/httpclient/api_oidc.go - About 1 hr to fix

                    Method OidcAPIService.CreateOidcDynamicClientExecute has 11 return statements (exceeds 4 allowed).
                    Open

                    func (a *OidcAPIService) CreateOidcDynamicClientExecute(r ApiCreateOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
                        var (
                            localVarHTTPMethod  = http.MethodPost
                            localVarPostBody    interface{}
                            formFiles           []formFile
                    Severity: Major
                    Found in internal/httpclient/api_oidc.go - About 1 hr to fix

                      Method OidcAPIService.CreateVerifiableCredentialExecute has 10 return statements (exceeds 4 allowed).
                      Open

                      func (a *OidcAPIService) CreateVerifiableCredentialExecute(r ApiCreateVerifiableCredentialRequest) (*VerifiableCredentialResponse, *http.Response, error) {
                          var (
                              localVarHTTPMethod  = http.MethodPost
                              localVarPostBody    interface{}
                              formFiles           []formFile
                      Severity: Major
                      Found in internal/httpclient/api_oidc.go - About 1 hr to fix

                        Method OidcAPIService.GetOidcDynamicClientExecute has 8 return statements (exceeds 4 allowed).
                        Open

                        func (a *OidcAPIService) GetOidcDynamicClientExecute(r ApiGetOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
                            var (
                                localVarHTTPMethod  = http.MethodGet
                                localVarPostBody    interface{}
                                formFiles           []formFile
                        Severity: Major
                        Found in internal/httpclient/api_oidc.go - About 50 mins to fix

                          Method OidcAPIService.DiscoverOidcConfigurationExecute has 8 return statements (exceeds 4 allowed).
                          Open

                          func (a *OidcAPIService) DiscoverOidcConfigurationExecute(r ApiDiscoverOidcConfigurationRequest) (*OidcConfiguration, *http.Response, error) {
                              var (
                                  localVarHTTPMethod  = http.MethodGet
                                  localVarPostBody    interface{}
                                  formFiles           []formFile
                          Severity: Major
                          Found in internal/httpclient/api_oidc.go - About 50 mins to fix

                            Method OidcAPIService.GetOidcUserInfoExecute has 8 return statements (exceeds 4 allowed).
                            Open

                            func (a *OidcAPIService) GetOidcUserInfoExecute(r ApiGetOidcUserInfoRequest) (*OidcUserInfo, *http.Response, error) {
                                var (
                                    localVarHTTPMethod  = http.MethodGet
                                    localVarPostBody    interface{}
                                    formFiles           []formFile
                            Severity: Major
                            Found in internal/httpclient/api_oidc.go - About 50 mins to fix

                              Method OidcAPIService.DeleteOidcDynamicClientExecute has 7 return statements (exceeds 4 allowed).
                              Open

                              func (a *OidcAPIService) DeleteOidcDynamicClientExecute(r ApiDeleteOidcDynamicClientRequest) (*http.Response, error) {
                                  var (
                                      localVarHTTPMethod = http.MethodDelete
                                      localVarPostBody   interface{}
                                      formFiles          []formFile
                              Severity: Major
                              Found in internal/httpclient/api_oidc.go - About 45 mins to fix

                                Method OidcAPIService.RevokeOidcSessionExecute has 6 return statements (exceeds 4 allowed).
                                Open

                                func (a *OidcAPIService) RevokeOidcSessionExecute(r ApiRevokeOidcSessionRequest) (*http.Response, error) {
                                    var (
                                        localVarHTTPMethod = http.MethodGet
                                        localVarPostBody   interface{}
                                        formFiles          []formFile
                                Severity: Major
                                Found in internal/httpclient/api_oidc.go - About 40 mins to fix

                                  Similar blocks of code found in 2 locations. Consider refactoring.
                                  Open

                                  func (a *OidcAPIService) SetOidcDynamicClientExecute(r ApiSetOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
                                      var (
                                          localVarHTTPMethod  = http.MethodPut
                                          localVarPostBody    interface{}
                                          formFiles           []formFile
                                  Severity: Major
                                  Found in internal/httpclient/api_oidc.go and 1 other location - About 1 day to fix
                                  internal/httpclient/api_o_auth2.go on lines 2399..2495

                                  Duplicated Code

                                  Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                  Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                  When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                  Tuning

                                  This issue has a mass of 840.

                                  We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                  The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                  If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                  See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                  Refactorings

                                  Further Reading

                                  Similar blocks of code found in 2 locations. Consider refactoring.
                                  Open

                                  func (a *OidcAPIService) CreateOidcDynamicClientExecute(r ApiCreateOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
                                      var (
                                          localVarHTTPMethod  = http.MethodPost
                                          localVarPostBody    interface{}
                                          formFiles           []formFile
                                  Severity: Major
                                  Found in internal/httpclient/api_oidc.go and 1 other location - About 1 day to fix
                                  internal/httpclient/api_o_auth2.go on lines 461..556

                                  Duplicated Code

                                  Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                  Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                  When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                  Tuning

                                  This issue has a mass of 797.

                                  We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                  The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                  If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                  See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                  Refactorings

                                  Further Reading

                                  Similar blocks of code found in 4 locations. Consider refactoring.
                                  Open

                                  func (a *OidcAPIService) GetOidcDynamicClientExecute(r ApiGetOidcDynamicClientRequest) (*OAuth2Client, *http.Response, error) {
                                      var (
                                          localVarHTTPMethod  = http.MethodGet
                                          localVarPostBody    interface{}
                                          formFiles           []formFile
                                  Severity: Major
                                  Found in internal/httpclient/api_oidc.go and 3 other locations - About 1 day to fix
                                  internal/httpclient/api_jwk.go on lines 510..590
                                  internal/httpclient/api_o_auth2.go on lines 907..987
                                  internal/httpclient/api_o_auth2.go on lines 1426..1506

                                  Duplicated Code

                                  Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                  Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                  When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                  Tuning

                                  This issue has a mass of 686.

                                  We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                  The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                  If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                  See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                  Refactorings

                                  Further Reading

                                  Similar blocks of code found in 4 locations. Consider refactoring.
                                  Open

                                  func (a *OidcAPIService) DiscoverOidcConfigurationExecute(r ApiDiscoverOidcConfigurationRequest) (*OidcConfiguration, *http.Response, error) {
                                      var (
                                          localVarHTTPMethod  = http.MethodGet
                                          localVarPostBody    interface{}
                                          formFiles           []formFile
                                  Severity: Major
                                  Found in internal/httpclient/api_oidc.go and 3 other locations - About 1 day to fix
                                  internal/httpclient/api_o_auth2.go on lines 2118..2197
                                  internal/httpclient/api_oidc.go on lines 670..749
                                  internal/httpclient/api_wellknown.go on lines 57..136

                                  Duplicated Code

                                  Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                  Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                  When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                  Tuning

                                  This issue has a mass of 643.

                                  We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                  The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                  If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                  See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                  Refactorings

                                  Further Reading

                                  Similar blocks of code found in 4 locations. Consider refactoring.
                                  Open

                                  func (a *OidcAPIService) GetOidcUserInfoExecute(r ApiGetOidcUserInfoRequest) (*OidcUserInfo, *http.Response, error) {
                                      var (
                                          localVarHTTPMethod  = http.MethodGet
                                          localVarPostBody    interface{}
                                          formFiles           []formFile
                                  Severity: Major
                                  Found in internal/httpclient/api_oidc.go and 3 other locations - About 1 day to fix
                                  internal/httpclient/api_o_auth2.go on lines 2118..2197
                                  internal/httpclient/api_oidc.go on lines 439..518
                                  internal/httpclient/api_wellknown.go on lines 57..136

                                  Duplicated Code

                                  Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                  Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                  When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                  Tuning

                                  This issue has a mass of 643.

                                  We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                  The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                  If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                  See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                  Refactorings

                                  Further Reading

                                  Similar blocks of code found in 4 locations. Consider refactoring.
                                  Open

                                  func (a *OidcAPIService) DeleteOidcDynamicClientExecute(r ApiDeleteOidcDynamicClientRequest) (*http.Response, error) {
                                      var (
                                          localVarHTTPMethod = http.MethodDelete
                                          localVarPostBody   interface{}
                                          formFiles          []formFile
                                  Severity: Major
                                  Found in internal/httpclient/api_oidc.go and 3 other locations - About 1 day to fix
                                  internal/httpclient/api_jwk.go on lines 290..360
                                  internal/httpclient/api_o_auth2.go on lines 591..661
                                  internal/httpclient/api_o_auth2.go on lines 802..872

                                  Duplicated Code

                                  Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                  Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                  When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                  Tuning

                                  This issue has a mass of 591.

                                  We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                  The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                  If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                  See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                  Refactorings

                                  Further Reading

                                  There are no issues that match your filters.

                                  Category
                                  Status