fixed failing unmarshaling & added comments

primaryHandler.go

@@ -318,9 +318,11 @@ func NewPrimaryHandler(logger *zap.Logger, manager device.Manager, v *viper.Vipe
 	}
 
 	// the secured variant of the device connect handler - compatible with v2 and v3
-	var failOpen = true
+	// default functionality is to allow for talaria to accept devices with or without authorization
+	// failOpen must be set to false in config in order to require authorization from any device trying to connect
+	failOpen := true
 	if v.IsSet(FailOpenConfigKey) {
-		err := v.UnmarshalKey(FailOpenConfigKey, failOpen)
+		err := v.UnmarshalKey(FailOpenConfigKey, &failOpen)
 		if err != nil {
 			logger.Error("failOpen parse failure", zap.Error(err))
 			return nil, errors.New("failed parsing FailOpen boolean")