Merge branch '600-improve-error-logging' into 'main'

Improve logging on errors

See merge request https://gitlab.com/gitlab-org/gitlab-shell/-/merge_requests/1156



Merged-by: Patrick Bajao <ebajao@gitlab.com>
Approved-by: Denys Mishunov <dmishunov@gitlab.com>
Approved-by: Patrick Bajao <ebajao@gitlab.com>
Reviewed-by: Patrick Bajao <ebajao@gitlab.com>
Co-authored-by: dec0dedd <michal28burzynski@gmail.com>

cmd/gitlab-shell-authorized-principals-check/main.go

@@ -42,7 +42,7 @@ func run() int {
 
 	config, err := config.NewFromDirExternal(executable.RootDir)
 	if err != nil {
-		_, _ = fmt.Fprintln(readWriter.ErrOut, "Failed to read config, exiting")
+		_, _ = fmt.Fprintln(readWriter.ErrOut, "Failed to read config, exiting:", err)
 		return 1
 	}
 

cmd/gitlab-shell/main.go

@@ -45,7 +45,7 @@ func main() {
 
 	config, err := config.NewFromDirExternal(executable.RootDir)
 	if err != nil {
-		fmt.Fprintln(readWriter.ErrOut, "Failed to read config, exiting")
+		fmt.Fprintln(readWriter.ErrOut, "Failed to read config, exiting:", err)
 		os.Exit(1)
 	}