sendnrw/netbird
2
0
Fork 0
mirror of https://github.com/netbirdio/netbird.git synced 2026-04-19 16:56:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix MySQL no-op update returning account not found in SaveAccountSettings

Browse Source
This commit is contained in:
Viktor Liu
2026-04-08 07:54:14 +02:00
parent fa777684c5
commit 86f1b53bd4
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
management/server/store/sql_store.go
View File

@@ -4077,9 +4077,10 @@ func (s *SqlStore) SaveAccountSettings(ctx context.Context, accountID string, se
return status.Errorf(status.Internal, "failed to save account settings to store") return status.Errorf(status.Internal, "failed to save account settings to store")
} }
if result.RowsAffected == 0 { // MySQL reports RowsAffected=0 for no-op updates where values don't change,
return status.NewAccountNotFoundError(accountID) // unlike SQLite/Postgres which report matched rows. Skip the check since the
} // caller (UpdateAccountSettings) already verified the account exists via
// GetAccountSettings with LockingStrengthUpdate.
return nil return nil
} }