Don't use Sprintf in Errorf for no reason

api/account/changepw.go

@@ -32,7 +32,7 @@ func ChangePW(uuid []byte, password string) error {
 	salt := make([]byte, ArgonSaltSize)
 	_, err := rand.Read(salt)
 	if err != nil {
-		return fmt.Errorf(fmt.Sprintf("failed to generate salt: %s", err))
+		return fmt.Errorf("failed to generate salt: %s", err)
 	}
 
 	err = db.UpdateAccountPassword(uuid, deriveArgon2IDKey([]byte(password), salt), salt)

api/account/register.go

@@ -20,6 +20,7 @@ package account
 import (
 	"crypto/rand"
 	"fmt"
+
 	"github.com/pagefaultgames/rogueserver/db"
 )
 
@@ -42,7 +43,7 @@ func Register(username, password string) error {
 	salt := make([]byte, ArgonSaltSize)
 	_, err = rand.Read(salt)
 	if err != nil {
-		return fmt.Errorf(fmt.Sprintf("failed to generate salt: %s", err))
+		return fmt.Errorf("failed to generate salt: %s", err)
 	}
 
 	err = db.AddAccountRecord(uuid, username, deriveArgon2IDKey([]byte(password), salt), salt)