strip newlines from check output

Signed-off-by: Justin Kolberg <[email protected]>

main.go

@@ -5,14 +5,15 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
-	"github.com/sensu/sensu-pagerduty-handler/pagerduty"
 	"log"
 	"net/url"
 	"os"
 	"regexp"
 	"strings"
 	"time"
 
+	"github.com/sensu/sensu-pagerduty-handler/pagerduty"
+
 	corev2 "github.com/sensu/core/v2"
 	"github.com/sensu/sensu-plugin-sdk/sensu"
 	"github.com/sensu/sensu-plugin-sdk/templates"
@@ -491,6 +492,11 @@ func getSummary(event *corev2.Event) (string, error) {
 
 func getDetails(event *corev2.Event) (details interface{}, err error) {
 	if len(config.detailsTemplate) > 0 {
+		// TODO: find a better solution to sanitize all event fields before
+		// template evaluation. Certain escape characters like '\n' cannot be
+		// parsed.
+		event.Check.Output = strings.ReplaceAll(event.Check.Output, "\n", " ")
+
 		detailsStr, err := templates.EvalTemplate("details", config.detailsTemplate, event)
 		if err != nil {
 			return "", fmt.Errorf("failed to evaluate template %s: %v", config.detailsTemplate, err)

main_test.go

@@ -166,9 +166,17 @@ func Test_GetDetailsTemplate(t *testing.T) {
 	event := corev2.FixtureEvent("foo", "bar")
 	config.detailsTemplate = "{{.Entity.Name}}-{{.Check.Name}}"
 
+	// Valid event
 	details, err := getDetails(event)
 	assert.Nil(t, err)
 	assert.Equal(t, "foo-bar", details)
+
+	// Invalid event with newline in check name
+	config.detailsFormat = "json"
+	config.detailsTemplate = "{\"Output\": \"`{{.Check.Output}}`\"}"
+	event.Check.Output = "bar\nxaz\n"
+	details, err = getDetails(event)
+	assert.Nil(t, err)
 }
 
 func Test_GetDetailsObj(t *testing.T) {