[exporter/syslog][chore] use consts for network protocol names (#32379)

use consts for network
protocol names, changes were suggested in
#31202 (comment)

**Testing:** unit tests, manual tests with syslog server

---------

Co-authored-by: Alex Boten <[email protected]>

Author: kasia-kujawa

exporter/syslogexporter/config.go

@@ -7,6 +7,7 @@ import (
 	"errors"
 	"strings"
 
+	"go.opentelemetry.io/collector/config/confignet"
 	"go.opentelemetry.io/collector/config/configretry"
 	"go.opentelemetry.io/collector/config/configtls"
 	"go.opentelemetry.io/collector/exporter/exporterhelper"
@@ -56,7 +57,8 @@ func (cfg *Config) Validate() error {
 		invalidFields = append(invalidFields, errInvalidEndpoint)
 	}
 
-	if strings.ToLower(cfg.Network) != "tcp" && strings.ToLower(cfg.Network) != "udp" {
+	cfg.Network = strings.ToLower(cfg.Network)
+	if cfg.Network != string(confignet.TransportTypeTCP) && cfg.Network != string(confignet.TransportTypeUDP) {
 		invalidFields = append(invalidFields, errUnsupportedNetwork)
 	}
 
@@ -80,7 +82,7 @@ func (cfg *Config) Validate() error {
 
 const (
 	// Syslog Network
-	DefaultNetwork = "tcp"
+	DefaultNetwork = string(confignet.TransportTypeTCP)
 	// Syslog Port
 	DefaultPort = 514
 	// Syslog Protocol

exporter/syslogexporter/exporter.go

@@ -9,6 +9,7 @@ import (
 	"fmt"
 	"strings"
 
+	"go.opentelemetry.io/collector/config/confignet"
 	"go.opentelemetry.io/collector/consumer/consumererror"
 	"go.opentelemetry.io/collector/exporter"
 	"go.opentelemetry.io/collector/exporter/exporterhelper"
@@ -25,10 +26,8 @@ type syslogexporter struct {
 }
 
 func initExporter(cfg *Config, createSettings exporter.CreateSettings) (*syslogexporter, error) {
-	cfg.Network = strings.ToLower(cfg.Network)
-
 	var loadedTLSConfig *tls.Config
-	if cfg.Network == "tcp" {
+	if cfg.Network == string(confignet.TransportTypeTCP) {
 		var err error
 		loadedTLSConfig, err = cfg.TLSSetting.LoadTLSConfig(context.Background())
 		if err != nil {
@@ -46,6 +45,7 @@ func initExporter(cfg *Config, createSettings exporter.CreateSettings) (*sysloge
 	s.logger.Info("Syslog Exporter configured",
 		zap.String("endpoint", cfg.Endpoint),
 		zap.String("protocol", cfg.Protocol),
+		zap.String("network", cfg.Network),
 		zap.Int("port", cfg.Port),
 	)
 
@@ -74,7 +74,7 @@ func newLogsExporter(
 }
 
 func (se *syslogexporter) pushLogsData(_ context.Context, logs plog.Logs) error {
-	batchMessages := strings.ToLower(se.config.Network) == "tcp"
+	batchMessages := se.config.Network == string(confignet.TransportTypeTCP)
 	var err error
 	if batchMessages {
 		err = se.exportBatch(logs)

exporter/syslogexporter/go.mod

@@ -5,6 +5,7 @@ go 1.21.0
 require (
 	github.com/stretchr/testify v1.9.0
 	go.opentelemetry.io/collector/component v0.99.0
+	go.opentelemetry.io/collector/config/confignet v0.99.0
 	go.opentelemetry.io/collector/config/configretry v0.99.0
 	go.opentelemetry.io/collector/config/configtls v0.99.0
 	go.opentelemetry.io/collector/exporter v0.99.0

exporter/syslogexporter/go.sum

@@ -10,6 +10,7 @@ import (
 	"strings"
 	"sync"
 
+	"go.opentelemetry.io/collector/config/confignet"
 	"go.uber.org/zap"
 )
 
@@ -78,7 +79,7 @@ func (s *sender) dial() error {
 		s.conn = nil
 	}
 	var err error
-	if s.tlsConfig != nil && s.network == "tcp" {
+	if s.tlsConfig != nil && s.network == string(confignet.TransportTypeTCP) {
 		s.conn, err = tls.Dial(s.network, s.addr, s.tlsConfig)
 	} else {
 		s.conn, err = net.Dial(s.network, s.addr)