Fix logs using incorrect environment variables

Hugo Müller-Downing · Hugo Müller-Downing · commit 45d16b80e35b · 2019-07-30T13:52:54.000+10:00
diff --git a/lib/providers/cloudflare/cloudflare.go b/lib/providers/cloudflare/cloudflare.go
@@ -20,27 +20,27 @@ var ZONE, HOST string
 func NewProvider() (providers.Provider, error) {
 	APIKEY := os.Getenv("CLOUDFLARE_APIKEY")
 	if APIKEY == "" {
-		log.Fatal("APIKEY env. variable is required")
+		log.Fatal("CLOUDFLARE_APIKEY env. variable is required")
 	}
 
 	ZONE = os.Getenv("CLOUDFLARE_ZONE")
 	if APIKEY == "" {
-		log.Fatal("ZONE env. variable is required")
+		log.Fatal("CLOUDFLARE_ZONE env. variable is required")
 	}
 
 	HOST = os.Getenv("CLOUDFLARE_HOST")
 	if HOST == "" {
-		log.Fatal("HOST env. variable is required")
+		log.Fatal("CLOUDFLARE_HOST env. variable is required")
 	}
 
 	EMAIL := os.Getenv("CLOUDFLARE_EMAIL")
 	if EMAIL == "" {
-		log.Fatal("EMAIL env. variable is required")
+		log.Fatal("CLOUDFLARE_EMAIL env. variable is required")
 	}
 
 	api, err := NewCloudflareClient(APIKEY, EMAIL, ZONE, HOST)
 	if err != nil {
-		panic(err)
+		return nil, err
 	}
 
 	provider := &Cloudflare{

Original file line number	Diff line number	Diff line change
`@@ -20,27 +20,27 @@ var ZONE, HOST string`
`20`	`20`	`func NewProvider() (providers.Provider, error) {`
`21`	`21`	`APIKEY := os.Getenv("CLOUDFLARE_APIKEY")`
`22`	`22`	`if APIKEY == "" {`
`23`		`- log.Fatal("APIKEY env. variable is required")`
	`23`	`+ log.Fatal("CLOUDFLARE_APIKEY env. variable is required")`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`ZONE = os.Getenv("CLOUDFLARE_ZONE")`
`27`	`27`	`if APIKEY == "" {`
`28`		`- log.Fatal("ZONE env. variable is required")`
	`28`	`+ log.Fatal("CLOUDFLARE_ZONE env. variable is required")`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`HOST = os.Getenv("CLOUDFLARE_HOST")`
`32`	`32`	`if HOST == "" {`
`33`		`- log.Fatal("HOST env. variable is required")`
	`33`	`+ log.Fatal("CLOUDFLARE_HOST env. variable is required")`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`EMAIL := os.Getenv("CLOUDFLARE_EMAIL")`
`37`	`37`	`if EMAIL == "" {`
`38`		`- log.Fatal("EMAIL env. variable is required")`
	`38`	`+ log.Fatal("CLOUDFLARE_EMAIL env. variable is required")`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`api, err := NewCloudflareClient(APIKEY, EMAIL, ZONE, HOST)`
`42`	`42`	`if err != nil {`
`43`		`- panic(err)`
	`43`	`+ return nil, err`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`provider := &Cloudflare{`