Merge pull request #2 from cgauge/map-issue

Use RWMutex for better concurrency

Author: abdala

Dockerfile

@@ -1,4 +1,4 @@
-FROM golang:1.17-alpine as build
+FROM golang:1.23-alpine AS build
 
 ARG CGO_ENABLED=0
 
@@ -12,14 +12,14 @@ RUN go mod download
 
 COPY . .
 
-FROM build as build-sidecar
+FROM build AS build-sidecar
 
 ARG CGO_ENABLED=0
 
 RUN go build -ldflags="-s -w" -o /cache-server main_container.go \
  && upx --best --lzma /cache-server
 
-FROM build as build-extension
+FROM build AS build-extension
 
 ARG CGO_ENABLED=0
 

secrets/cache.go

@@ -27,7 +27,7 @@ type CacheData struct {
 	CacheExpiry time.Time
 }
 
-var mutex = &sync.Mutex{}
+var mutex = &sync.RWMutex{}
 
 func IsExpired(cacheExpiry time.Time) bool {
 	return cacheExpiry.Before(time.Now())
@@ -48,9 +48,14 @@ func GetCacheExpiry() time.Time {
 }
 
 func GetSecretCache(name string, refresh string) string {
+	mutex.RLock()
 	secret := secretCache[name]
+	mutex.RUnlock()
+
+	println("GetSecretCache: ", name)
 
 	if IsExpired(secret.CacheData.CacheExpiry) || refresh == "1" {
+		println("GetSecretCache: refresh")
 		mutex.Lock()
 		secretCache[name] = Secret{
 			CacheData: CacheData{