fix(config): Prevent loading malformed yaml

If the YAML config contains malformed content the
error is ignored silently. To prevent this, we
only proceed with default settings loading if the
config file doesn't exist. On the contrary, the
error is raised.

internal/bootstrap/bootstrap.go

@@ -180,7 +180,7 @@ func (f *App) Run(args []string) error {
 	}
 
 	log.Infof("bootstrapping with pid %d. Version: %s", os.Getpid(), version.Get())
-	log.Infof("configuration dump %s", cfg.Print())
+	log.Infof("rendering config flags... %s", cfg.Print())
 
 	// build the filter from the CLI argument. If we got
 	// a valid expression the filter is attached to the

internal/bootstrap/config.go

@@ -22,28 +22,33 @@ import (
 	"github.com/rabbitstack/fibratus/pkg/config"
 	"github.com/rabbitstack/fibratus/pkg/util/log"
 	"github.com/sirupsen/logrus"
+	"os"
 )
 
 // InitConfigAndLogger initializes the configuration and sets up the logger.
 // We allow continuing with the initialization process even if the config file
 // loading fails. In this situation, the default config flag values are used
 // to tweak any of the internal behaviours.
 func InitConfigAndLogger(cfg *config.Config) error {
-	isLoaded := cfg.TryLoadFile(cfg.File()) == nil
+	err := cfg.TryLoadFile(cfg.File())
+	notExists := os.IsNotExist(err)
+	if err != nil && !notExists {
+		return err
+	}
 	if err := cfg.Init(); err != nil {
 		return err
 	}
-	if isLoaded {
+	if err == nil {
 		if err := cfg.Validate(); err != nil {
 			return err
 		}
 	}
 	if err := log.InitFromConfig(cfg.Log, "fibratus.log"); err != nil {
 		return err
 	}
-	if !isLoaded {
-		logrus.Warnf("unable to load configuration "+
-			"from %s file. Falling back to default "+
+	if notExists {
+		logrus.Infof("configuration file "+
+			"%s not found. Continuing with default "+
 			"settings...", cfg.File())
 	}
 	return nil