add default readtimeout and writetimeout

Signed-off-by: jiangyufang <[email protected]>

Makefile.Common

@@ -48,6 +48,7 @@ $(TOOLS_BIN_NAMES): $(TOOLS_BIN_DIR) $(TOOLS_MOD_DIR)/go.mod
 
 .PHONY: test
 test:
+# $(GOTEST) $(GOTEST_OPT) -run TestSetDefaultConfig
 	$(GOTEST) $(GOTEST_OPT) ./...
 
 .PHONY: test-with-cover

receiver/otlpreceiver/config.go

@@ -8,6 +8,7 @@ import (
 	"fmt"
 	"net/url"
 	"path"
+	"time"
 
 	"go.opentelemetry.io/collector/component"
 	"go.opentelemetry.io/collector/config/configgrpc"
@@ -17,8 +18,9 @@ import (
 
 const (
 	// Protocol values.
-	protoGRPC = "protocols::grpc"
-	protoHTTP = "protocols::http"
+	protoGRPC      = "protocols::grpc"
+	protoHTTP      = "protocols::http"
+	defaultTimeout = 60 * time.Second
 )
 
 type HTTPConfig struct {
@@ -83,6 +85,14 @@ func (cfg *Config) Unmarshal(conf *confmap.Conf) error {
 		if cfg.HTTP.LogsURLPath, err = sanitizeURLPath(cfg.HTTP.LogsURLPath); err != nil {
 			return err
 		}
+		//set default  server config value
+		if cfg.HTTP.ReadTimeout == 0 {
+			cfg.HTTP.ReadTimeout = defaultTimeout
+		}
+
+		if cfg.HTTP.WriteTimeout == 0 {
+			cfg.HTTP.WriteTimeout = defaultTimeout
+		}
 	}
 
 	return nil

receiver/otlpreceiver/config_test.go

@@ -138,6 +138,17 @@ func TestUnmarshalConfig(t *testing.T) {
 
 }
 
+func TestSetDefaultConfig(t *testing.T) {
+	cm, err := confmaptest.LoadConf(filepath.Join("testdata", "config.yaml"))
+	require.NoError(t, err)
+	factory := NewFactory()
+	cfg := factory.CreateDefaultConfig()
+	assert.NoError(t, cm.Unmarshal(&cfg))
+	assert.Equal(t, 60*time.Second, (cfg.(*Config)).HTTP.WriteTimeout)
+	assert.Equal(t, 60*time.Second, (cfg.(*Config)).HTTP.WriteTimeout)
+
+}
+
 func TestUnmarshalConfigUnix(t *testing.T) {
 	cm, err := confmaptest.LoadConf(filepath.Join("testdata", "uds.yaml"))
 	require.NoError(t, err)