ensure context is set before spawning goroutines using it (#2569)

* ensure context is set before spawning goroutines using it * ensure waiting gRPC calls aren't notified before context is set Co-authored-by: Robert Bailey <[email protected]>
googleforgames · May 11, 2022 · 015822a · 015822a
1 parent 9d669d5
commit 015822a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/pkg/sdkserver/sdkserver.go b/pkg/sdkserver/sdkserver.go
@@ -199,6 +199,9 @@ func (s *SDKServer) Run(ctx context.Context) error {
 	if !cache.WaitForCacheSync(ctx.Done(), s.gameServerSynced) {
 		return errors.New("failed to wait for caches to sync")
 	}
+
+	// need this for streaming gRPC commands
+	s.ctx = ctx
 	// we have the gameserver details now
 	s.gsWaitForSync.Done()
 
@@ -262,8 +265,6 @@ func (s *SDKServer) Run(ctx context.Context) error {
 	}()
 	defer s.server.Close() // nolint: errcheck
 
-	// need this for streaming gRPC commands
-	s.ctx = ctx
 	s.workerqueue.Run(ctx, 1)
 	return nil
 }