Changeset 173 in code for trunk/db.go

Timestamp:

Mar 27, 2020, 9:38:38 PM (5 years ago)

Author:

contact

Message:

Stop accessing user data in downstreamConn.authenticate

This becomes racy once user.Password is updated on-the-fly.

File:

• trunk/db.go (modified) (1 diff)

trunk/db.go

@@ -105 +105 @@
 
         return users, nil
+}
+
+func (db *DB) GetUser(username string) (*User, error) {
+        db.lock.RLock()
+        defer db.lock.RUnlock()
+
+        user := &User{Username: username}
+
+        var password *string
+        row := db.db.QueryRow("SELECT password FROM User WHERE username = ?", username)
+        if err := row.Scan(&password); err != nil {
+                return nil, err
+        }
+        user.Password = fromStringPtr(password)
+        return user, nil
 }