synced/coder
1
0
Fork 0
mirror of https://github.com/coder/coder.git synced 2025-07-08 11:39:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: add mutex to MockAuditor export to prevent race (#5189)

Browse Source 
See: https://github.com/coder/coder/actions/runs/3575201153/jobs/6011435900.
This commit is contained in:
Kyle Carberry
2022-11-30 17:25:30 +01:00
committed by GitHub
parent 41f10e7b69
commit 2a73362026
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
coderd/audit/audit.go
View File

@ -2,6 +2,7 @@ package audit
import ( import (
"context" "context"
"sync"
"github.com/coder/coder/coderd/database" "github.com/coder/coder/coderd/database"
) )
@ -30,10 +31,13 @@ func NewMock() *MockAuditor {
} }
type MockAuditor struct { type MockAuditor struct {
mutex sync.Mutex
AuditLogs []database.AuditLog AuditLogs []database.AuditLog
} }
func (a *MockAuditor) Export(_ context.Context, alog database.AuditLog) error { func (a *MockAuditor) Export(_ context.Context, alog database.AuditLog) error {
a.mutex.Lock()
defer a.mutex.Unlock()
a.AuditLogs = append(a.AuditLogs, alog) a.AuditLogs = append(a.AuditLogs, alog)
return nil return nil
} }