Update escapes to match newest action

There is a fix in actions toolkit: actions/toolkit#302, apply it here as well

core/command.go

@@ -15,6 +15,14 @@ const (
 var (
 	stdout       io.Writer = os.Stdout
 	stdoutSetter sync.Mutex
+	dataEscapes  = map[string]string{
+		"\r": "%0D",
+		"\n": "%0A",
+	}
+	escapes = map[string]string{
+		":": "%3A",
+		",": "%2C",
+	}
 )
 
 func SetStdout(w io.Writer) {
@@ -53,25 +61,20 @@ func (c *command) String() string {
 	return s + cmdString + escape(c.message)
 }
 
+func escapePatterns(v string, replacementsArg ...map[string]string) string {
+	v = strings.Replace(v, "%", "%25", -1)
+	for _, replacements := range replacementsArg {
+		for pattern, replacement := range replacements {
+			v = strings.Replace(v, pattern, replacement, -1)
+		}
+	}
+	return v
+}
+
 func escapeData(v string) string {
-	return strings.Replace(
-		strings.Replace(
-			v,
-			"\r", "%0D", -1,
-		),
-		"\n", "%0A", -1,
-	)
+	return escapePatterns(v, dataEscapes)
 }
 
 func escape(v string) string {
-	return strings.Replace(
-		strings.Replace(
-		strings.Replace(
-			escapeData(v),
-			"]", "%5D", -1,
-		),
-		";", "%3B", -1,
-	),
-	",", "%2C",-1,
-)
+	return escapePatterns(v, escapes, dataEscapes)
 }

core/command_internal_test.go

@@ -2,6 +2,8 @@ package core
 
 import (
 	"bytes"
+	"regexp"
+	"strings"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -21,8 +23,13 @@ func TestIssueCommand(t *testing.T) {
 	b := bytes.NewBuffer(nil)
 	stdout = b
 	IssueCommand("hello", map[string]string{
-		"some":  "a\n\rvalue,];",
+		"some":  "a\n\rvalue,:%",
 		"other": "value",
-	}, "some\r\nmessage")
-	assert.Equal(t, "::hello some=a%0A%0Dvalue%2C%5D%3B,other=value::some%0D%0Amessage\n", b.String())
+	}, "some\r\n%message")
+	assert.Contains(t, b.String(), "some=a%0A%0Dvalue%2C%3A%25")
+	assert.Contains(t, b.String(), "other=value")
+	assert.Regexp(t, regexp.MustCompile("::some%0D%0A%25message\n$"), b.String())
+	assert.Regexp(t, regexp.MustCompile("^::hello "), b.String())
+	assert.Len(t, strings.Split(b.String(), ","), 2)
+
 }