adds some logging if there is an error reading config files (#118)

ChrisJBurns · web-flow · commit 45da2119bd16 · 2025-04-08T08:17:58.000+03:00
diff --git a/pkg/client/config.go b/pkg/client/config.go
@@ -17,6 +17,7 @@ import (
 	"gopkg.in/yaml.v3"
 
 	"github.com/stacklok/toolhive/pkg/config"
+	"github.com/stacklok/toolhive/pkg/logger"
 	"github.com/stacklok/toolhive/pkg/transport/ssecommon"
 )
 
@@ -155,6 +156,10 @@ func FindClientConfigs() ([]ConfigFile, error) {
 			clientConfig.Editor = pe.Editor
 			configs = append(configs, clientConfig)
 		}
+
+		if err != nil {
+			logger.Log.Error(fmt.Sprintf("Error reading client config file %s: %v", pe.Path, err))
+		}
 	}
 
 	return configs, nil

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ import (`
`17`	`17`	`"gopkg.in/yaml.v3"`
`18`	`18`
`19`	`19`	`"github.com/stacklok/toolhive/pkg/config"`
	`20`	`+ "github.com/stacklok/toolhive/pkg/logger"`
`20`	`21`	`"github.com/stacklok/toolhive/pkg/transport/ssecommon"`
`21`	`22`	`)`
`22`	`23`
`@@ -155,6 +156,10 @@ func FindClientConfigs() ([]ConfigFile, error) {`
`155`	`156`	`clientConfig.Editor = pe.Editor`
`156`	`157`	`configs = append(configs, clientConfig)`
`157`	`158`	`}`
	`159`	`+`
	`160`	`+ if err != nil {`
	`161`	`+ logger.Log.Error(fmt.Sprintf("Error reading client config file %s: %v", pe.Path, err))`
	`162`	`+ }`
`158`	`163`	`}`
`159`	`164`
`160`	`165`	`return configs, nil`