Merge pull request #574 from openziti/fix-refresh-tight-loop

fix refresh tight loop
openziti · May 31, 2024 · a0ebf80 · a0ebf80
2 parents 871efb6 + 109282b
commit a0ebf80
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,10 @@
+# Release notes 0.23.38
+
+## Issues Fixed and Dependency Updates
+
+* github.com/openziti/sdk-golang: [v0.23.37 -> v0.23.38](https://github.com/openziti/sdk-golang/compare/v0.23.37...v0.23.38)
+    * [Issue #573](https://github.com/openziti/sdk-golang/issues/573) - api session refresh spins in a tight loop if there is no current api session
+
 # Release notes 0.23.37
 
 ## Issues Fixed and Dependency Updates

diff --git a/ziti/sdkinfo/build_info.go b/ziti/sdkinfo/build_info.go
diff --git a/ziti/ziti.go b/ziti/ziti.go
@@ -796,7 +796,11 @@ func (context *ContextImpl) runRefreshes() {
 			apiSession := context.CtrlClt.GetCurrentApiSession()
 
 			if apiSession == nil {
-				pfxlog.Logger().Warn("could not refresh api session, current api session is nil")
+				pfxlog.Logger().Warn("could not refresh api session, current api session is nil, authenticating")
+				if err := context.Authenticate(); err != nil {
+					pfxlog.Logger().WithError(err).Error("failed to authenticate")
+				}
+				refreshAt = time.Now().Add(5 * time.Second)
 				continue
 			}