From 00441f9844544f025625506eb38406e9e658a43e Mon Sep 17 00:00:00 2001 From: "Gustavo \"Guz\" L de Mello" Date: Tue, 10 Jun 2025 14:40:07 -0300 Subject: [PATCH] feat(repo,user): dont use transactions on select queries --- repository/user.go | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/repository/user.go b/repository/user.go index 727ad3d..0bc0e85 100644 --- a/repository/user.go +++ b/repository/user.go @@ -96,16 +96,12 @@ func (repo *UserRepository) GetByUsername(username string) (model.User, error) { log := repo.log.With(slog.String("username", username), slog.String("query", q)) log.DebugContext(repo.ctx, "Querying user") - row := tx.QueryRowContext(r.ctx, q, sql.Named("username", username)) + row := repo.db.QueryRowContext(repo.ctx, q, sql.Named("username", username)) var password_hash, dateCreated, dateUpdated string - if err = row.Scan(&username, &password_hash, &dateCreated, &dateUpdated); err != nil { - return model.User{}, err - } - - if err := tx.Commit(); err != nil { - log.ErrorContext(r.ctx, "Failed to commit transaction", slog.String("error", err.Error())) - return model.User{}, err + err := row.Scan(&username, &password_hash, &dateCreated, &dateUpdated) + if err != nil { + return model.User{}, errors.Join(ErrExecuteQuery, err) } passwd, err := base64.URLEncoding.DecodeString(password_hash)