fix args in db queries

pashcovich · Dec 2, 2022 · 5eefd1b · 5eefd1b
1 parent bc7f26f
commit 5eefd1b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/openvpn-user.go b/openvpn-user.go
@@ -10,7 +10,7 @@ import (
 )
 
 const (
-	version = "1.0.8"
+	version = "1.0.9"
 )
 
 var (

diff --git a/src/commands.go b/src/commands.go
@@ -213,7 +213,7 @@ func (oUser *OpenvpnUser) RegisterOtpApplication(username, totp string) (string,
 				return "", authErr
 			}
 			if authOk {
-				_, err := oUser.Database.Exec("UPDATE users SET app_configured = 1 WHERE username = $2")
+				_, err := oUser.Database.Exec("UPDATE users SET app_configured = 1 WHERE username = $1", username)
 				if err != nil {
 					return "", err
 				}
@@ -232,7 +232,7 @@ func (oUser *OpenvpnUser) ResetOtpApplication(username string) (string, error) {
 			return "", appErr
 		}
 		if appConfigured {
-			_, err := oUser.Database.Exec("UPDATE users SET app_configured = 0 WHERE username = $2")
+			_, err := oUser.Database.Exec("UPDATE users SET app_configured = 0 WHERE username = $1", username)
 			if err != nil {
 				return "", err
 			}
@@ -263,6 +263,7 @@ func (oUser *OpenvpnUser) IsSecondFactorEnabled(username string) (bool, error) {
 		if u.name == username {
 			return u.appConfigured, nil
 		}
+
 		return false, checkAppError
 	}
 	return false, userIsNotActiveError