fix/BCF-2497: refactoring out redundant query implementations

smartcontractkit · Sep 5, 2023 · 00d005d · 00d005d
1 parent 505f367
commit 00d005d
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/core/sessions/orm.go b/core/sessions/orm.go
@@ -112,11 +112,12 @@ func (o *orm) AuthorizedUserWithSession(sessionID string) (User, error) {
 		return User{}, ErrUserSessionExpired
 	}
 
-	if err := o.q.Get(&user, "SELECT * FROM users WHERE lower(email) = lower($1)", foundSession.Email); err != nil {
+	user, err := o.findUser(foundSession.Email)
+	if err != nil {
 		return User{}, errors.Wrap(err, "no matching user for provided session email")
 	}
 
-	err := o.q.Transaction(func(tx pg.Queryer) error {
+	err = o.q.Transaction(func(tx pg.Queryer) error {
 		// Session valid and tied to user, update last_used
 		_, err := tx.Exec("UPDATE sessions SET last_used = now() WHERE id = $1 AND last_used + $2 >= now()", sessionID, o.sessionDuration)
 		if err != nil {