From 2a73362026cfbcff86e9dca2684a04d32f44f867 Mon Sep 17 00:00:00 2001 From: Kyle Carberry Date: Wed, 30 Nov 2022 17:25:30 +0100 Subject: [PATCH] fix: add mutex to MockAuditor export to prevent race (#5189) See: https://github.com/coder/coder/actions/runs/3575201153/jobs/6011435900. --- coderd/audit/audit.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/coderd/audit/audit.go b/coderd/audit/audit.go index ca343123e9..1342d27a38 100644 --- a/coderd/audit/audit.go +++ b/coderd/audit/audit.go @@ -2,6 +2,7 @@ package audit import ( "context" + "sync" "github.com/coder/coder/coderd/database" ) @@ -30,10 +31,13 @@ func NewMock() *MockAuditor { } type MockAuditor struct { + mutex sync.Mutex AuditLogs []database.AuditLog } func (a *MockAuditor) Export(_ context.Context, alog database.AuditLog) error { + a.mutex.Lock() + defer a.mutex.Unlock() a.AuditLogs = append(a.AuditLogs, alog) return nil }