Merge pull request #4 from percipia/cdata

Send AppArgs for Set/Export/Push via the body of the SendMessage execute command

README.md

@@ -13,7 +13,7 @@ eslgo was written from the ground up in idiomatic Go for use in our production p
 go get github.com/percipia/eslgo
 ```
 ```
-github.com/percipia/eslgo v1.3.1
+github.com/percipia/eslgo v1.3.2
 ```
 
 ## Overview

command/call/execute.go

@@ -18,13 +18,14 @@ import (
 )
 
 type Execute struct {
-	UUID    string
-	AppName string
-	AppArgs string
-	AppUUID string
-	Loops   int
-	Sync    bool
-	SyncPri bool
+	UUID      string
+	AppName   string
+	AppArgs   string
+	AppUUID   string
+	Loops     int
+	Sync      bool
+	SyncPri   bool
+	ForceBody bool
 }
 
 // Helper to call Execute with Set since it is commonly used
@@ -44,11 +45,12 @@ type Push Set
 
 func (s Set) buildMessage(app string) string {
 	e := Execute{
-		UUID:    s.UUID,
-		AppName: app,
-		AppArgs: fmt.Sprintf("%s=%s", s.Key, s.Value),
-		Sync:    s.Sync,
-		SyncPri: s.SyncPri,
+		UUID:      s.UUID,
+		AppName:   app,
+		AppArgs:   fmt.Sprintf("%s=%s", s.Key, s.Value),
+		Sync:      s.Sync,
+		SyncPri:   s.SyncPri,
+		ForceBody: true,
 	}
 	return e.BuildMessage()
 }
@@ -84,7 +86,7 @@ func (e *Execute) BuildMessage() string {
 	}
 
 	// According to documentation that is the max header length
-	if len(e.AppArgs) > 2048 {
+	if len(e.AppArgs) > 2048 || e.ForceBody {
 		sendMsg.Headers.Set("content-type", "text/plain")
 		sendMsg.Headers.Set("content-length", strconv.Itoa(len(e.AppArgs)))
 		sendMsg.Body = e.AppArgs

command/call/execute_test.go

@@ -24,19 +24,28 @@ Execute-App-Name: playback
 Loops: 1`, "\n", "\r\n")
 	TestSetMessage = strings.ReplaceAll(`sendmsg none
 Call-Command: execute
-Execute-App-Arg: hello=world
+Content-Length: 11
+Content-Type: text/plain
 Execute-App-Name: set
-Loops: 1`, "\n", "\r\n")
+Loops: 1
+
+hello=world`, "\n", "\r\n")
 	TestExportMessage = strings.ReplaceAll(`sendmsg none
 Call-Command: execute
-Execute-App-Arg: hello=world
+Content-Length: 11
+Content-Type: text/plain
 Execute-App-Name: export
-Loops: 1`, "\n", "\r\n")
+Loops: 1
+
+hello=world`, "\n", "\r\n")
 	TestPushMessage = strings.ReplaceAll(`sendmsg none
 Call-Command: execute
-Execute-App-Arg: hello=world
+Content-Length: 11
+Content-Type: text/plain
 Execute-App-Name: push
-Loops: 1`, "\n", "\r\n")
+Loops: 1
+
+hello=world`, "\n", "\r\n")
 )
 
 func TestExecute_BuildMessage(t *testing.T) {