(ci) Fix some more issues found with DeepSource

Author: christianarty

pkg/oauth/oauth.go

@@ -38,6 +38,9 @@ const (
 
 	// HTTP client timeout for API calls.
 	httpClientTimeout = 30 * time.Second
+
+	// Read header timeout for API calls.
+	readHeaderTimeout = 3 * time.Second
 )
 
 var defaultScopes = []string{
@@ -217,7 +220,8 @@ func performOAuthFlow(config *OAuthConfig, httpTimeout time.Duration, openBrowse
 	errChan := make(chan error, 1)
 
 	server := &http.Server{
-		Addr: defaultPort,
+		Addr:              defaultPort,
+		ReadHeaderTimeout: readHeaderTimeout,
 		Handler: http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 			if r.URL.Path == callbackPath {
 				code := r.URL.Query().Get("code")

pkg/oauth/oauth_test.go

@@ -332,7 +332,10 @@ func TestPerformOAuthFlow_ErrorCases(t *testing.T) {
 		}
 
 		// Start a server on the same port to cause a conflict
-		conflictServer := &http.Server{Addr: defaultPort}
+		conflictServer := &http.Server{
+			Addr:              defaultPort,
+			ReadHeaderTimeout: readHeaderTimeout,
+		}
 		go func() {
 			_ = conflictServer.ListenAndServe()
 		}()

pkg/utils/storage_test.go

@@ -10,8 +10,6 @@ import (
 )
 
 func TestFileSystemStorage(t *testing.T) {
-	t.Parallel()
-
 	t.Run("creates directory and saves file", func(t *testing.T) {
 		// Create temporary directory
 		tempDir := t.TempDir()
@@ -80,8 +78,6 @@ func TestFileSystemStorage(t *testing.T) {
 }
 
 func TestStorageOperations(t *testing.T) {
-	t.Parallel()
-
 	t.Run("storage save and load operations", func(t *testing.T) {
 		storage := &mockStorage{}