Merge pull request #121 from hpe-storage/logger_windows_append_CRLF

logger:  Windows Notepad requires CRLF and not just LF

Author: NimbleMike

logger/logger.go

@@ -1,4 +1,4 @@
-// Copyright 2019 Hewlett Packard Enterprise Development LP
+// Copyright 2020 Hewlett Packard Enterprise Development LP
 
 package logger
 
@@ -332,6 +332,17 @@ func (hook *FileHook) Fire(entry *log.Entry) error {
 		return err
 	}
 
+	// For Windows only, insert '/r' in front of any tailing '/n'.  Windows text files end
+	// lines with CRLF while other platforms just end with LF.
+	if runtime.GOOS == "windows" {
+		for i := len(lineBytes) - 1; i > 0; i-- {
+			if (lineBytes[i] != '\n') || (i > 0 && lineBytes[i-1] == '\r') {
+				break
+			}
+			lineBytes = append(lineBytes[:i], append([]byte{'\r'}, lineBytes[i:]...)...)
+		}
+	}
+
 	hook.logWriter.Write(lineBytes)
 	return nil
 }