fix: panic on read local cache error (#727)

gorse-io · Jul 15, 2023 · 9b5b39a · 9b5b39a
1 parent 5c2160f
commit 9b5b39a
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/.github/workflows/build_release.yml b/.github/workflows/build_release.yml
@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Set up Go 1.20.x
-        uses: actions/setup-go@v2
+        uses: actions/setup-go@v4
         with:
           go-version: 1.20.x
         id: go

diff --git a/.github/workflows/build_test.yml b/.github/workflows/build_test.yml
@@ -74,7 +74,7 @@ jobs:
         unzip ~/.gorse/download/criteo.zip -d ~/.gorse/dataset
 
     - name: Set up Go 1.20.x
-      uses: actions/setup-go@v2
+      uses: actions/setup-go@v4
       with:
         go-version: 1.20.x
       id: go
@@ -141,7 +141,7 @@ jobs:
         retry-all: true
 
     - name: Set up Go 1.20.x
-      uses: actions/setup-go@v2
+      uses: actions/setup-go@v4
       with:
         go-version: 1.20.x
       id: go
@@ -171,7 +171,7 @@ jobs:
 
     steps:
       - name: Set up Go 1.20.x
-        uses: actions/setup-go@v2
+        uses: actions/setup-go@v4
         with:
           go-version: 1.20.x
         id: go
@@ -188,7 +188,7 @@ jobs:
     name: lint
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/setup-go@v2
+      - uses: actions/setup-go@v4
         with:
           go-version: 1.20.x
       - uses: actions/checkout@v2

diff --git a/server/server.go b/server/server.go
@@ -79,7 +79,7 @@ func (s *Server) Serve() {
 			log.Logger().Info("no cache file found, create a new one", zap.String("path", s.cacheFile))
 		} else {
 			log.Logger().Error("failed to connect local store", zap.Error(err),
-				zap.String("path", state.path))
+				zap.String("path", s.cacheFile))
 		}
 	}
 	if state.ServerName == "" {

diff --git a/worker/worker.go b/worker/worker.go
@@ -352,7 +352,7 @@ func (w *Worker) Serve() {
 				log.Logger().Info("no cache file found, create a new one", zap.String("path", state.path))
 			} else {
 				log.Logger().Error("failed to load persist state", zap.Error(err),
-					zap.String("path", state.path))
+					zap.String("path", w.cacheFile))
 			}
 		}
 		if state.WorkerName == "" {