Don't error when dropping a non-existent user

alphagov · Dec 2, 2024 · 0c02cfa · 0c02cfa
1 parent 3fa1640
commit 0c02cfa
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/sqlengine/mysql_engine.go b/sqlengine/mysql_engine.go
@@ -168,7 +168,7 @@ func (d *MySQLEngine) DropUser(bindingID string) error {
 		return err
 	}
 
-	dropUserStatement := "DROP USER `" + username + "`@`%`;"
+	dropUserStatement := "DROP USER IF EXISTS `" + username + "`@`%`;"
 	logger.Debug("drop-user", lager.Data{"statement": dropUserStatement})
 
 	_, err := d.db.Exec(dropUserStatement)
@@ -186,7 +186,7 @@ func (d *MySQLEngine) DropUser(bindingID string) error {
 		return err
 	}
 
-	dropUserStatement = "DROP USER IF EXISTS`" + username + "`@`%`;"
+	dropUserStatement = "DROP USER IF EXISTS `" + username + "`@`%`;"
 	logger.Debug("drop-user", lager.Data{"statement": dropUserStatement})
 
 	_, err = d.db.Exec(dropUserStatement)