Merge pull request #588 from sipcapture/revert-587-alert-autofix-10

Revert "Fix code scanning alert no. 10: Database query built from user-controlled sources"
sipcapture · Jan 7, 2025 · 5d9073d · 5d9073d
2 parents 3103633 + e4a98a3
commit 5d9073d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/data/service/userSettings.go b/data/service/userSettings.go
@@ -177,7 +177,7 @@ func (ss *UserSettingsService) Get(userObject *model.TableUserSettings, UserName
 
 	if err := ss.Session.Debug().
 		Table("user_settings").
-		Where("guid = ? AND username = ?", userObject.GUID, UserName).Find(&data).Error; err != nil {
+		Where(sqlWhere).Find(&data).Error; err != nil {
 		return data, err
 	}
 	return data, nil
@@ -197,7 +197,7 @@ func (ss *UserSettingsService) Delete(userObject *model.TableUserSettings, UserN
 
 	if err := ss.Session.Debug().
 		Table("user_settings").
-		Where("guid = ? AND username = ?", userObject.GUID, UserName).
+		Where(sqlWhere).
 		Delete(model.TableUserSettings{}).Error; err != nil {
 		return err
 	}
@@ -220,7 +220,7 @@ func (ss *UserSettingsService) Update(userObject *model.TableUserSettings, UserN
 		Table("user_settings").
 		Debug().
 		Model(&model.TableUserSettings{}).
-		Where("guid = ? AND username = ?", userObject.GUID, UserName).Update(userObject).Error; err != nil {
+		Where(sqlWhere).Update(userObject).Error; err != nil {
 		return err
 	}
 	return nil