fix(envd): replace time.Sleep with ticker in ScanAndBroadcast for prompt shutdown (#3374)

Replace `time.Sleep` with `time.NewTicker` so `scanExit` and the tick
interval are waited on simultaneously in a single `select`.

`Destroy()` now unblocks the goroutine instantly, regardless of how far
into the current interval it is.

Fixes: #3356
Co-authored-by: Petr Vaněk <petr.vanek@e2b.dev>

Author: AdaAibaby

packages/envd/internal/port/scan.go

@@ -43,6 +43,9 @@ func (s *Scanner) Unsubscribe(sub *ScannerSubscriber) {
 
 // ScanAndBroadcast starts scanning open TCP ports and broadcasts every open port to all subscribers.
 func (s *Scanner) ScanAndBroadcast() {
+	ticker := time.NewTicker(s.period)
+	defer ticker.Stop()
+
 	for {
 		// tcp monitors both ipv4 and ipv6 connections.
 		processes, _ := net.Connections("tcp")
@@ -52,8 +55,7 @@ func (s *Scanner) ScanAndBroadcast() {
 		select {
 		case <-s.scanExit:
 			return
-		default:
-			time.Sleep(s.period)
+		case <-ticker.C:
 		}
 	}
 }

packages/envd/pkg/version.go

@@ -1,3 +1,3 @@
 package pkg
 
-const Version = "0.6.10"
+const Version = "0.6.11"