Replace log.Print calls with slog.InfoContext

Azure · Dec 17, 2024 · 5fd6028 · 5fd6028
1 parent d61ad30
commit 5fd6028
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 7 deletions.
diff --git a/cli/azd/internal/vsrpc/server.go b/cli/azd/internal/vsrpc/server.go
@@ -8,7 +8,6 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
-	"log"
 	"log/slog"
 	"net"
 	"net/http"
@@ -96,7 +95,7 @@ func (s *Server) Serve(l net.Listener) error {
 func serveRpc(w http.ResponseWriter, r *http.Request, handlers map[string]Handler) {
 	c, err := upgrader.Upgrade(w, r, nil)
 	if err != nil {
-		log.Print("upgrade:", err)
+		slog.InfoContext(r.Context(), "error upgrading connect", "err", err)
 		return
 	}
 	defer c.Close()

diff --git a/cli/azd/main.go b/cli/azd/main.go
@@ -237,7 +237,7 @@ func fetchLatestVersion(version chan<- semver.Version) {
 
 	// If we don't have a cached version we can use, fetch one (and cache it)
 	if cachedLatestVersion == nil {
-		log.Print("fetching latest version information for update check")
+		slog.InfoContext(context.TODO(), "fetching latest version information for update check")
 		req, err := http.NewRequest(http.MethodGet, "https://aka.ms/azure-dev/versions/cli/latest", nil)
 		if err != nil {
 			slog.InfoContext(context.TODO(), "failed to create request object, skipping update check", "err", err)

diff --git a/cli/azd/pkg/exec/command_runner.go b/cli/azd/pkg/exec/command_runner.go
@@ -6,7 +6,7 @@ import (
 	"errors"
 	"fmt"
 	"io"
-	"log"
+	"log/slog"
 	"os"
 	"os/exec"
 	"path/filepath"
@@ -304,7 +304,7 @@ func (l *logBuilder) Write(debug bool, sensitiveArgsData []string) {
 		}
 	}
 
-	log.Print(msg.String())
+	slog.InfoContext(context.TODO(), msg.String())
 }
 
 // newCmdTree creates a `CmdTree`, optionally using a shell appropriate for windows

diff --git a/cli/azd/pkg/oneauth/oneauth_windows.go b/cli/azd/pkg/oneauth/oneauth_windows.go
@@ -38,7 +38,7 @@ import (
 	"context"
 	_ "embed"
 	"fmt"
-	"log"
+	"log/slog"
 	"os"
 	"path/filepath"
 	"strings"
@@ -54,7 +54,7 @@ import (
 
 //export goLog
 func goLog(s *C.char) {
-	log.Print(C.GoString(s))
+	slog.InfoContext(context.TODO(), C.GoString(s))
 }
 
 // Supported indicates whether this build includes OneAuth integration.