register additional gob types to support arrays and dicts in post props (#8412)

2 files changed, 40 insertions, 5 deletions

diff --git a/plugin/rpcplugin/api.go b/plugin/rpcplugin/api.go
index 5b5b11a62..d87f65b55 100644
--- a/plugin/rpcplugin/api.go
+++ b/plugin/rpcplugin/api.go
@@ -610,4 +610,6 @@ func ConnectAPI(conn io.ReadWriteCloser, muxer *Muxer) *RemoteAPI {
 
 func init() {
 	gob.Register([]*model.SlackAttachment{})
+	gob.Register([]interface{}{})
+	gob.Register(map[string]interface{}{})
 }
diff --git a/plugin/rpcplugin/api_test.go b/plugin/rpcplugin/api_test.go
index 145ec9005..7fe7a0ff9 100644
--- a/plugin/rpcplugin/api_test.go
+++ b/plugin/rpcplugin/api_test.go
@@ -72,11 +72,6 @@ func TestAPI(t *testing.T) {
 
 	testPost := &model.Post{
 		Message: "hello",
-		Props: map[string]interface{}{
-			"attachments": []*model.SlackAttachment{
-				&model.SlackAttachment{},
-			},
-		},
 	}
 
 	testAPIRPC(&api, func(remote plugin.API) {
@@ -244,3 +239,41 @@ func TestAPI(t *testing.T) {
 		assert.Nil(t, err)
 	})
 }
+
+func TestAPI_GobRegistration(t *testing.T) {
+	keyValueStore := &plugintest.KeyValueStore{}
+	api := plugintest.API{Store: keyValueStore}
+	defer api.AssertExpectations(t)
+
+	testAPIRPC(&api, func(remote plugin.API) {
+		api.On("CreatePost", mock.AnythingOfType("*model.Post")).Return(func(p *model.Post) (*model.Post, *model.AppError) {
+			p.Id = "thepostid"
+			return p, nil
+		}).Once()
+		_, err := remote.CreatePost(&model.Post{
+			Message: "hello",
+			Props: map[string]interface{}{
+				"attachments": []*model.SlackAttachment{
+					&model.SlackAttachment{
+						Actions: []*model.PostAction{
+							&model.PostAction{
+								Integration: &model.PostActionIntegration{
+									Context: map[string]interface{}{
+										"foo":  "bar",
+										"foos": []interface{}{"bar", "baz", 1, 2},
+										"foo_map": map[string]interface{}{
+											"1": "bar",
+											"2": 2,
+										},
+									},
+								},
+							},
+						},
+						Timestamp: 1,
+					},
+				},
+			},
+		})
+		require.Nil(t, err)
+	})
+}