ory-am/hydra

File persister_consent.go has 646 lines of code (exceeds 500 allowed). Consider refactoring.

Open

// Copyright © 2022 Ory Corp
// SPDX-License-Identifier: Apache-2.0

package sql

Persister has 36 methods (exceeds 20 allowed). Consider refactoring.

Open

func (p *Persister) RevokeSubjectConsentSession(ctx context.Context, user string) error {
    ctx, span := p.r.Tracer(ctx).Tracer().Start(ctx, "persistence.sql.RevokeSubjectConsentSession")
    defer span.End()

    return p.Transaction(ctx, p.revokeConsentSession("consent_challenge_id IS NOT NULL AND subject = ?", user))

Method Persister.ConfirmLoginSession has 7 return statements (exceeds 4 allowed).

Open

func (p *Persister) ConfirmLoginSession(ctx context.Context, loginSession *flow.LoginSession) error {
    ctx, span := p.r.Tracer(ctx).Tracer().Start(ctx, "persistence.sql.ConfirmLoginSession")
    defer span.End()

    loginSession.NID = p.NetworkID(ctx)

Method Persister.VerifyAndInvalidateConsentRequest has 5 return statements (exceeds 4 allowed).

Open

func (p *Persister) VerifyAndInvalidateConsentRequest(ctx context.Context, verifier string) (*flow.AcceptOAuth2ConsentRequest, error) {
    ctx, span := p.r.Tracer(ctx).Tracer().Start(ctx, "persistence.sql.VerifyAndInvalidateConsentRequest")
    defer span.End()

    f, err := flowctx.Decode[flow.Flow](ctx, p.r.FlowCipher(), verifier, flowctx.AsConsentVerifier)

Method Persister.mySQLConfirmLoginSession has 5 return statements (exceeds 4 allowed).

Open

func (p *Persister) mySQLConfirmLoginSession(ctx context.Context, session *flow.LoginSession) error {
    err := sqlcon.HandleError(p.Connection(ctx).Create(session))
    if err == nil {
        return nil
    }

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

Open

func (p *Persister) ListUserAuthenticatedClientsWithBackChannelLogout(ctx context.Context, subject, sid string) ([]client.Client, error) {
    ctx, span := p.r.Tracer(ctx).Tracer().Start(ctx, "persistence.sql.ListUserAuthenticatedClientsWithBackChannelLogout")
    defer span.End()
    return p.listUserAuthenticatedClients(ctx, subject, sid, "back")
}

• Read up
  Read up

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

Open

func (p *Persister) ListUserAuthenticatedClientsWithFrontChannelLogout(ctx context.Context, subject, sid string) ([]client.Client, error) {
    ctx, span := p.r.Tracer(ctx).Tracer().Start(ctx, "persistence.sql.ListUserAuthenticatedClientsWithFrontChannelLogout")
    defer span.End()

    return p.listUserAuthenticatedClients(ctx, subject, sid, "front")

• Read up
  Read up