Better error reporting about configuration file

- Better errors in config.go
- Do not print useless timestamps in cmd/senpai/main.go
- Let os.UserConfigDir() and senpai.NewApp() call panic on error since
  they both should not fail.

1 files changed, 8 insertions, 2 deletions

diff --git a/config.go b/config.go
index fcd4cc3..1d8d79a 100644
--- a/config.go
+++ b/config.go
@@ -2,6 +2,7 @@ package senpai
 
 import (
 	"errors"
+	"fmt"
 	"io/ioutil"
 
 	"gopkg.in/yaml.v2"
@@ -28,6 +29,9 @@ type Config struct {
 
 func ParseConfig(buf []byte) (cfg Config, err error) {
 	err = yaml.Unmarshal(buf, &cfg)
+	if err != nil {
+		return cfg, err
+	}
 	if cfg.Addr == "" {
 		return cfg, errors.New("addr is required")
 	}
@@ -54,10 +58,12 @@ func LoadConfigFile(filename string) (cfg Config, err error) {
 
 	buf, err = ioutil.ReadFile(filename)
 	if err != nil {
-		return
+		return cfg, fmt.Errorf("failed to read the file: %s", err)
 	}
 
 	cfg, err = ParseConfig(buf)
-
+	if err != nil {
+		return cfg, fmt.Errorf("invalid content found in the file: %s", err)
+	}
 	return
 }