Adds support for buildpack log level environment variable 'BP_LOG_LEV…

…EL' which currently accepts 'INFO' (default) or 'DEBUG'

Signed-off-by: David O'Sullivan <davidos@vmware.com>

poet/logger.go

@@ -57,13 +57,25 @@ func NewLoggerWithOptions(writer io.Writer, options ...Option) Logger {
 func NewLogger(writer io.Writer) Logger {
 	var options []Option
 
-	if _, ok := os.LookupEnv("BP_DEBUG"); ok {
-		options = append(options, WithDebug(writer))
-	}
+	// check for presence and value of log level environment variable
+	options = LogLevel(options, writer)
 
 	return NewLoggerWithOptions(writer, options...)
 }
 
+func LogLevel(options []Option, writer io.Writer) []Option {
+
+	// Check for older log level env variable
+	_, dbSet := os.LookupEnv("BP_DEBUG")
+
+	// Then check for common buildpack log level env variable - if either are set to DEBUG/true, enable Debug Writer
+	if level, ok := os.LookupEnv("BP_LOG_LEVEL"); (ok && strings.ToLower(level) == "debug") || dbSet {
+
+		options = append(options, WithDebug(writer))
+	}
+	return options
+}
+
 // Debug formats using the default formats for its operands and writes to the configured debug writer. Spaces are added
 // between operands when neither is a string.
 func (l Logger) Debug(a ...interface{}) {

poet/logger_test.go

@@ -64,6 +64,21 @@ func testLogger(t *testing.T, context spec.G, it spec.S) {
 		})
 	})
 
+	context("with BP_LOG_LEVEL set to DEBUG", func() {
+		it.Before(func() {
+			Expect(os.Setenv("BP_LOG_LEVEL", "DEBUG")).To(Succeed())
+			l = poet.NewLogger(b)
+		})
+
+		it.After(func() {
+			Expect(os.Unsetenv("BP_LOG_LEVEL")).To(Succeed())
+		})
+
+		it("configures debug", func() {
+			Expect(l.IsDebugEnabled()).To(BeTrue())
+		})
+	})
+
 	context("with debug disabled", func() {
 		it.Before(func() {
 			l = poet.NewLoggerWithOptions(b)