[CRE-954] Extra logging limits in WASM Module

+ set a missing "mode" field on executor

Author: bolekk

pkg/workflows/wasm/host/execution.go

@@ -27,6 +27,8 @@ type execution[T any] struct {
 	mode                 sdkpb.Mode
 	donSeed              int64
 	nodeSeed             int64
+	donLogCount          uint32
+	nodeLogCount         uint32
 }
 
 // callCapAsync async calls a capability by placing execution results onto a
@@ -139,6 +141,32 @@ func (e *execution[T]) awaitSecrets(ctx context.Context, acr *sdkpb.AwaitSecrets
 }
 
 func (e *execution[T]) log(caller *wasmtime.Caller, ptr int32, ptrlen int32) {
+	switch e.mode {
+	case sdkpb.Mode_MODE_DON:
+		e.donLogCount++
+		if e.donLogCount == e.module.cfg.MaxLogCountDONMode {
+			e.module.cfg.Logger.Errorf("max log count for don mode reached: %d - all subsequent logs will be dropped", e.donLogCount)
+		}
+		if e.donLogCount > e.module.cfg.MaxLogCountDONMode {
+			// silently drop to avoid spamming logs
+			return
+		}
+	case sdkpb.Mode_MODE_NODE:
+		e.nodeLogCount++
+		if e.nodeLogCount == e.module.cfg.MaxLogCountNodeMode {
+			e.module.cfg.Logger.Errorf("max log count for node mode reached: %d - all subsequent logs will be dropped", e.nodeLogCount)
+		}
+		if e.nodeLogCount > e.module.cfg.MaxLogCountNodeMode {
+			// silently drop to avoid spamming logs
+			return
+		}
+	}
+
+	if ptrlen > int32(e.module.cfg.MaxLogLenBytes) {
+		e.module.cfg.Logger.Errorf("log message too long: %d - dropping", ptrlen)
+		return
+	}
+
 	b, innerErr := wasmRead(caller, ptr, ptrlen)
 	if innerErr != nil {
 		e.module.cfg.Logger.Errorf("error calling log: %s", innerErr)

pkg/workflows/wasm/host/module.go

@@ -41,6 +41,9 @@ var (
 	defaultMaxCompressedBinarySize   = 20 * 1024 * 1024  // 20 MB
 	defaultMaxDecompressedBinarySize = 100 * 1024 * 1024 // 100 MB
 	defaultMaxResponseSizeBytes      = 5 * 1024 * 1024   // 5 MB
+	defaultMaxLogLenBytes            = 1024 * 1024       // 1 MB
+	defaultMaxLogCountDONMode        = 10_000
+	defaultMaxLogCountNodeMode       = 10_000
 	ResponseBufferTooSmall           = "response buffer too small"
 )
 
@@ -62,6 +65,10 @@ type ModuleConfig struct {
 	MaxDecompressedBinarySize uint64
 	MaxResponseSizeBytes      uint64
 
+	MaxLogLenBytes      uint32
+	MaxLogCountDONMode  uint32
+	MaxLogCountNodeMode uint32
+
 	// Labeler is used to emit messages from the module.
 	Labeler custmsg.MessageEmitter
 
@@ -183,6 +190,15 @@ func NewModule(modCfg *ModuleConfig, binary []byte, opts ...func(*ModuleConfig))
 	if modCfg.MaxResponseSizeBytes == 0 {
 		modCfg.MaxResponseSizeBytes = uint64(defaultMaxResponseSizeBytes)
 	}
+	if modCfg.MaxLogLenBytes == 0 {
+		modCfg.MaxLogLenBytes = uint32(defaultMaxLogLenBytes)
+	}
+	if modCfg.MaxLogCountDONMode == 0 {
+		modCfg.MaxLogCountDONMode = uint32(defaultMaxLogCountDONMode)
+	}
+	if modCfg.MaxLogCountNodeMode == 0 {
+		modCfg.MaxLogCountNodeMode = uint32(defaultMaxLogCountNodeMode)
+	}
 
 	// Take the max of the min and the configured max memory mbs.
 	// We do this because Go requires a minimum of 16 megabytes to run,
@@ -529,6 +545,7 @@ func runWasm[I, O proto.Message](
 		secretsResponses:    map[int32]<-chan *secretsResponse{},
 		module:              m,
 		executor:            helper,
+		mode:                sdkpb.Mode_MODE_DON,
 		donSeed:             donSeed,
 		nodeSeed:            int64(rand.Uint64()),
 	}

pkg/workflows/wasm/host/standard_test.go

@@ -29,6 +29,7 @@ import (
 	"github.com/smartcontractkit/chainlink-common/pkg/capabilities/v2/protoc/pkg/test_capabilities/basicaction"
 	"github.com/smartcontractkit/chainlink-common/pkg/capabilities/v2/protoc/pkg/test_capabilities/basictrigger"
 	"github.com/smartcontractkit/chainlink-common/pkg/capabilities/v2/protoc/pkg/test_capabilities/nodeaction"
+	"github.com/smartcontractkit/chainlink-common/pkg/logger"
 	"github.com/smartcontractkit/chainlink-protos/cre/go/sdk"
 	"github.com/smartcontractkit/chainlink-protos/cre/go/values"
 	valuespb "github.com/smartcontractkit/chainlink-protos/cre/go/values/pb"
@@ -234,6 +235,44 @@ func TestStandardLogging(t *testing.T) {
 	runWithBasicTrigger(t, mockExecutionHelper)
 }
 
+func TestStandardLoggingWithLimits(t *testing.T) {
+	t.Parallel()
+	mockExecutionHelper := NewMockExecutionHelper(t)
+	mockExecutionHelper.EXPECT().GetWorkflowExecutionID().Return("id")
+	mockExecutionHelper.EXPECT().GetNodeTime().RunAndReturn(func() time.Time {
+		return time.Now()
+	}).Maybe()
+	mockExecutionHelper.EXPECT().GetDONTime().RunAndReturn(func() (time.Time, error) {
+		return time.Now(), nil
+	}).Maybe()
+
+	logs := []string{}
+	mockExecutionHelper.EXPECT().EmitUserLog(mock.Anything).RunAndReturn(func(s string) error {
+		logs = append(logs, s)
+		return nil
+	})
+
+	trigger := &basictrigger.Outputs{CoolOutput: anyTestTriggerValue}
+	executeRequest := triggerExecuteRequest(t, 0, trigger)
+	cfg := &ModuleConfig{
+		Logger:              logger.Test(t),
+		IsUncompressed:      true,
+		MaxLogLenBytes:      20,
+		MaxLogCountDONMode:  3,
+		MaxLogCountNodeMode: 3,
+	}
+
+	m := makeTestModuleByName(t, "logging_limits", cfg)
+	_, err := m.Execute(t.Context(), executeRequest, mockExecutionHelper)
+	require.NoError(t, err)
+
+	// allowed 3 logs max, one of which got rejected because it was too long
+	// so expect 2 logs to be emitted
+	require.Equal(t, 2, len(logs))
+	require.Equal(t, "short log 1", logs[0])
+	require.Equal(t, "short log 3", logs[1])
+}
+
 func TestStandardMultipleTriggers(t *testing.T) {
 	t.Parallel()
 	m := makeTestModule(t)

pkg/workflows/wasm/host/standard_tests/logging_limits/main_wasip1.go

@@ -0,0 +1,18 @@
+package main
+
+import (
+	"github.com/smartcontractkit/chainlink-common/pkg/workflows/wasm/host/internal/rawsdk"
+)
+
+func main() {
+	request := rawsdk.GetRequest()
+	msg := []byte("short log 1")
+	rawsdk.Log(rawsdk.BufferToPointerLen(msg))
+	msg = []byte("super duper excessively long log 2") // exceeding 20 byte limit set in the test
+	rawsdk.Log(rawsdk.BufferToPointerLen(msg))
+	msg = []byte("short log 3")
+	rawsdk.Log(rawsdk.BufferToPointerLen(msg))
+	msg = []byte("short log 4")
+	rawsdk.Log(rawsdk.BufferToPointerLen(msg))
+	rawsdk.SendResponse(request.Config)
+}