Be explicit about crlsMutex's use

Co-authored-by: Miciah Dashiel Butler Masters <[email protected]>
openshift · Sep 17, 2024 · e7b4fc2 · e7b4fc2
1 parent e6243d4
commit e7b4fc2
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/pkg/router/crl/crl.go b/pkg/router/crl/crl.go
@@ -68,8 +68,10 @@ var (
 	// CABundleFilename is the fully qualified path to the currently in use CA bundle.
 	CABundleFilename = filepath.Join(mtlsLatestSymlink, caBundleBasename)
 	// crlsUpdated is true when all CRLs have been successfully updated, and false when there are missing CRLs.
+	// You must take crlsMutex before using crlsUpdated.  
 	crlsUpdated = false
-	crlsMutex   = sync.Mutex{}
+	// crlsMutex protects crlsUpdated.  
+	crlsMutex = sync.Mutex{}
 )
 
 // authorityKeyIdentifier is a certificate's authority key identifier.